Skip to content

Udpating cudaq-guide skill and adding qiskit-to-cudaq skill#347

Open
sacpis wants to merge 1 commit into
NVIDIA:mainfrom
sacpis:update_cudaq_guide_skill
Open

Udpating cudaq-guide skill and adding qiskit-to-cudaq skill#347
sacpis wants to merge 1 commit into
NVIDIA:mainfrom
sacpis:update_cudaq_guide_skill

Conversation

@sacpis

@sacpis sacpis commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Udpating cudaq-guide skill and adding qiskit-to-cudaq skill

@mitchdz

mitchdz commented Jul 14, 2026

Copy link
Copy Markdown

/nv-skills

@mosheabr mosheabr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks Sachin, the content itself looks like solid work, but this PR can't land here. The skills/ directories in this catalog are synced from each product's source repo every hour, so direct edits here get reverted by the next sync, and the content integrity check is failing because the changed and added files don't carry signature refreshes.

The right path for both changes. Make the cudaq-guide updates and add qiskit-to-cudaq in the CUDA-Q source repo, run the signing pipeline there for both skills (that also regenerates the skill card and benchmark for the new skill), and add a qiskit-to-cudaq entry to components.d/cuda-q.yml in this repo. The sync then brings the signed content in automatically. One more thing worth doing in the same pass, cudaq-guide currently has unsigned changes at source that the sync keeps holding back, so a re-sign there clears that too.

Happy to walk you through the steps if useful. Closing the content changes out of this PR, the components.d registration line can happen here once the source side is signed.

Signed-off-by: Sachin Pisal <spisal@nvidia.com>
@sacpis
sacpis force-pushed the update_cudaq_guide_skill branch from fb7ec6b to 4b9943d Compare July 14, 2026 20:37
@sacpis
sacpis requested a review from mosheabr July 14, 2026 20:37
@sacpis

sacpis commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

The right path for both changes. Make the cudaq-guide updates and add qiskit-to-cudaq in the CUDA-Q source repo, run the signing pipeline there for both skills (that also regenerates the skill card and benchmark for the new skill

This is already done in cuda-quantum repo with the PR #4812.

add a qiskit-to-cudaq entry to components.d/cuda-q.yml in this repo.

Added with commit 4b9943d.

Thank you @mosheabr for detailed guidance here.

@mosheabr

Copy link
Copy Markdown
Collaborator

Thanks Sachin, the PR is now exactly the right shape, one registration entry, and the source PR route is correct. One gap remains before this can merge. The signing pipeline hasn't actually run on qiskit-to-cudaq in cuda-quantum. The skill there has SKILL.md, evals, and references, but no skill.oms.sig, skill-card.md, or BENCHMARK.md, and those are required before the catalog will sync it. Trigger the pipeline on a PR in cuda-quantum (comment /nvskills-ci on it) and the missing artifacts get generated and committed. While you're at it, cudaq-guide has content changes at source that were never re-signed, which is why the catalog sync keeps holding its updates back. The same pipeline run can clear that too. Once the artifacts are in cuda-quantum main, this PR merges as is.

@sacpis

sacpis commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Thanks Sachin, the PR is now exactly the right shape, one registration entry, and the source PR route is correct. One gap remains before this can merge. The signing pipeline hasn't actually run on qiskit-to-cudaq in cuda-quantum. The skill there has SKILL.md, evals, and references, but no skill.oms.sig, skill-card.md, or BENCHMARK.md, and those are required before the catalog will sync it. Trigger the pipeline on a PR in cuda-quantum (comment /nvskills-ci on it) and the missing artifacts get generated and committed. While you're at it, cudaq-guide has content changes at source that were never re-signed, which is why the catalog sync keeps holding its updates back. The same pipeline run can clear that too. Once the artifacts are in cuda-quantum main, this PR merges as is.

Thanks @mosheabr. Have requested help from @mitchdz to run the command on the existing PR.

@sacpis

sacpis commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

Hey @mosheabr, Is this good to go now? We ran /nvskill-ci on a PR in CUDA-Q. Do we need to wait for it to be merged?

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