Skip to content

[features/run] Add return to output logging pass.#2582

Merged
schweitzpgi merged 6 commits intoNVIDIA:features/runfrom
schweitzpgi:ch-fr-return.to.output
Feb 6, 2025
Merged

[features/run] Add return to output logging pass.#2582
schweitzpgi merged 6 commits intoNVIDIA:features/runfrom
schweitzpgi:ch-fr-return.to.output

Conversation

@schweitzpgi
Copy link
Collaborator

Add test.
Add struct test.

Description

Add test.
Add struct test.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
@schweitzpgi schweitzpgi force-pushed the ch-fr-return.to.output branch from ebd48b3 to cdf366b Compare February 3, 2025 22:58
github-actions bot pushed a commit that referenced this pull request Feb 4, 2025
Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
github-actions bot pushed a commit that referenced this pull request Feb 4, 2025
explicitly since it appears GCC-12 isn't smart enough to
know how to compile them.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
github-actions bot pushed a commit that referenced this pull request Feb 5, 2025
@NVIDIA NVIDIA deleted a comment from copy-pr-bot bot Feb 5, 2025
@schweitzpgi
Copy link
Collaborator Author

The codecov "error" is bogus. We haven't integrated everything.

@schweitzpgi schweitzpgi enabled auto-merge (squash) February 6, 2025 14:18
Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
Copy link
Collaborator

@khalatepradnya khalatepradnya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽

@schweitzpgi schweitzpgi merged commit d8a4142 into NVIDIA:features/run Feb 6, 2025
51 of 59 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 6, 2025
@schweitzpgi schweitzpgi deleted the ch-fr-return.to.output branch February 6, 2025 20:03
annagrin pushed a commit to annagrin/cuda-quantum that referenced this pull request Jun 17, 2025
* Start to add return to output logging pass.
Add test.
Add struct test.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>

* Workaround overly excitable warnings in GCC 12.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>

* Make the labels like the design doc.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>

* Update the CHECK lines.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>

* Wrap all string building terms in std::string constructors
explicitly since it appears GCC-12 isn't smart enough to
know how to compile them.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>

---------

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
Signed-off-by: Anna Gringauze <agringauze@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants