-
Notifications
You must be signed in to change notification settings - Fork 19
Fix 217: Merge function for Qasm Modules #261
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
👋 Hey there! It looks like the changelog might need an update. Please take a moment to edit the
|
|
🚫 Logo Change Blocked Changes to Why is this blocked?
What should you do?
Files changed in this PR: |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
🚫 Logo Change Blocked Changes to Why is this blocked?
What should you do?
Files changed in this PR: |
|
🚫 Logo Change Blocked Changes to Why is this blocked?
What should you do?
Files changed in this PR: |
|
Closing in favour of #267 |
Summary of changes
Fixes issue 217 by adding a merge function, so two
QasmModuleobjects can be combined into one. The merge function unrolls both modules and creates a single qubit declaration. This function appends the secondQasmModuleobject with the correct index offsets and returns a Qasm3Module.The
test_merge.pyfile is added, which has a few examples of merging two modules, using the merge function.