Skip to content

Fix kron order used in initial state construction in example notebook#3574

Merged
1tnguyen merged 2 commits intoNVIDIA:mainfrom
1tnguyen:tnguyen/fix-dynamics_intro_1-nb
Nov 5, 2025
Merged

Fix kron order used in initial state construction in example notebook#3574
1tnguyen merged 2 commits intoNVIDIA:mainfrom
1tnguyen:tnguyen/fix-dynamics_intro_1-nb

Conversation

@1tnguyen
Copy link
Collaborator

@1tnguyen 1tnguyen commented Nov 4, 2025

Description

  • Fix wrong kron order in the notebook example. Also, edit a wrong comment.

  • Upload new reference result images.

Resolved #3573

Signed-off-by: Thien Nguyen <thiennguyen@nvidia.com>
@1tnguyen 1tnguyen requested a review from sacpis November 4, 2025 20:42
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 @1tnguyen.

github-actions bot pushed a commit that referenced this pull request Nov 4, 2025
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

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

github-actions bot pushed a commit that referenced this pull request Nov 4, 2025
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

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

@1tnguyen 1tnguyen merged commit 2db20ce into NVIDIA:main Nov 5, 2025
363 of 370 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 5, 2025
khalatepradnya pushed a commit to khalatepradnya/cuda-quantum that referenced this pull request Nov 17, 2025
Signed-off-by: Thien Nguyen <thiennguyen@nvidia.com>
@bettinaheim bettinaheim added the documentation Improvements or additions to documentation label Nov 25, 2025
@bettinaheim bettinaheim added this to the release 0.13.0 milestone Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Questions about Jaynes-Cummings Model simulation in dynamics_intro_1.ipynb

3 participants