Skip to content

change imports and structure to accomadate new name of deltakit-stim#246

Draft
SeanDLawlor wants to merge 2 commits into
mainfrom
users/SeanDLawlor/propagate_deltakitlestim_into_repository
Draft

change imports and structure to accomadate new name of deltakit-stim#246
SeanDLawlor wants to merge 2 commits into
mainfrom
users/SeanDLawlor/propagate_deltakitlestim_into_repository

Conversation

@SeanDLawlor
Copy link
Copy Markdown
Collaborator

@SeanDLawlor SeanDLawlor commented Mar 30, 2026

🔗 Closed Issues

Link any issues closed or comments addressed by this PR.
"Closes #X" will close issue X automatically when the PR is merged.


📝 Description

propagate/prepare for deltakit-stim name change


🚦 Status

Build locally and everything seems to work


🛠️ Future Work

N/A


➕️ Additional Information


🧾 Release Note

Currently just s single commit PR - "change lest import to deltakit-stim"

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 30, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link
Copy Markdown

File Coverage Missing
All files 85%
deltakit-core/src/deltakit_core/decoding_graphs/_dem_parsing.py 99% 241
deltakit-decode/src/deltakit_decode/noise_sources/_stim_noise_sources.py 94% 117-119 127 287

Minimum allowed coverage is 95%

Generated by 🐒 cobertura-action against 90f8846

@SeanDLawlor SeanDLawlor marked this pull request as ready for review March 31, 2026 08:28
@SeanDLawlor SeanDLawlor requested review from nelimee and removed request for nelimee March 31, 2026 08:28
nelimee
nelimee previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Collaborator

@nelimee nelimee left a comment

Choose a reason for hiding this comment

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

One comment about the import guards, but except that you are good to go.

Note for myself: we should likely change all import stim to import deltakit_stim eventually, in all packages.

@SeanDLawlor SeanDLawlor force-pushed the users/SeanDLawlor/propagate_deltakitlestim_into_repository branch from 90f8846 to 51ca10c Compare March 31, 2026 12:08
@nelimee nelimee marked this pull request as draft April 1, 2026 16:34
@nelimee
Copy link
Copy Markdown
Collaborator

nelimee commented Apr 1, 2026

Marking as draft, coming back at this PR early next week.

Copy link
Copy Markdown
Collaborator

@rolandriver rolandriver left a comment

Choose a reason for hiding this comment

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

Thanks for this @SeanDLawlor. There are other places where this should be corrected, for instance see here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants