Skip to content

Conversation

khalatepradnya
Copy link
Collaborator

@khalatepradnya khalatepradnya commented Sep 26, 2025

This PR refactors how sample results are created from QIR output logs in the QuantinuumServerHelper, moving logic from the implementation file into a reusable method in the base server helper class.

Attempt to break up large PR #3448 into smaller chunks.

Signed-off-by: Pradnya Khalate <pkhalate@nvidia.com>
Copy link
Collaborator

@sacpis sacpis left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @khalatepradnya. Just one comment.

github-actions bot pushed a commit that referenced this pull request Sep 27, 2025
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Sep 29, 2025
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Sep 30, 2025
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

Copy link
Collaborator

@1tnguyen 1tnguyen left a comment

Choose a reason for hiding this comment

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

Thanks @khalatepradnya!

github-actions bot pushed a commit that referenced this pull request Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 2025

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

@khalatepradnya khalatepradnya enabled auto-merge (squash) October 1, 2025 02:16
@khalatepradnya khalatepradnya merged commit a7c8ee3 into NVIDIA:main Oct 1, 2025
128 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 2025

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

@khalatepradnya khalatepradnya deleted the refactoring-1 branch October 1, 2025 16:06
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.

3 participants