Skip to content

Fix licensing issues in example #1618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 9 commits into
base: amd-mainline
Choose a base branch
from
Open

Fix licensing issues in example #1618

wants to merge 9 commits into from

Conversation

@CatherineMoore
Copy link
Collaborator

@CatherineMoore CatherineMoore commented Oct 31, 2025

Technical Details

Remove SimpleTimer.h due to invalid license.
Recode use of timer routines using in tsp.cpp.

  1. Remove counting_iterator.h due to licensing issues
    2. Update Makefile to use c++-20
    3. Remove counting_iterator references in call to transform_reduce

CatherineMoore and others added 9 commits September 7, 2025 20:09
@CatherineMoore
Add AMD Copyright and MIT license designation.
2952f22
Merge pull request #272 from AMD-Lightning-Internal/amd/dev/dsalinas/…
6829323 
…lic-4

Add AMD Copyright and MIT license designation.
@estewart08
[smoke-limbo] - Default to AOMP clang instead of GCC for cmake invoca…
4440aa7 
…tion (#1502) (#100)

Running into an issue where the gcc on some testers seems incomplete
or ccache is misconfigured. Also fix the clean target to remove build
dir and not use 'git clean' because the tests are not always in a git
repo when tested.
Merge pull request #316 from AMD-Lightning-Internal/amd/dev/dsalinas/…
68c4a00 
…rocm72_cp

[smoke-limbo] - Default to AOMP clang instead of GCC for cmake invoca…
@CatherineMoore
Add MIT license.
2b7443e
@CatherineMoore
Merge pull request #414 from AMD-Lightning-Internal/amd/dev/catmoore/…
18e6496 
…cp-3

Add MIT license.
@CatherineMoore @ronlieb
Add AMD copyright and MIT license designation.
113a64b
@CatherineMoore
Remove SimpleTimer.h due to invalid license.
2d9b460 
Recode use of timer routines using <chrono> in tsp.cpp.
@CatherineMoore
1. Remove counting_iterator.h due to licensing issues
a431dc5 
2. Update Makefile to use c++-20
3. Remove counting_iterator references in call to transform_reduce
@CatherineMoore CatherineMoore requested review from ronlieb and removed request for ronlieb October 31, 2025 17:30
@ronlieb ronlieb requested a review from saiislam as a code owner November 6, 2025 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ronlieb ronlieb ronlieb approved these changes

@estewart08 estewart08 Awaiting requested review from estewart08 estewart08 is a code owner

@gregrodgers gregrodgers Awaiting requested review from gregrodgers gregrodgers is a code owner

@saiislam saiislam Awaiting requested review from saiislam saiislam is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@CatherineMoore @ronlieb @estewart08