Remove deprecated qubit auto-alignment - #1043
Conversation
|
@edoaltamura @adekusar-drl @oscar-wallis @OkuyanBoga looking forward to your reviews on the given PR .Happy to make further changes and developments . |
|
@oscar-wallis @OkuyanBoga @edoaltamura @adekusar-drl looking forward to your review on PR and response from your side. |
|
Hey @oscar-wallis @OkuyanBoga @edoaltamura @adekusar-drl looking forward to your review on PR and response from your side. Can you please review this PR and I'm happy to make further changes if needed. |
|
Hi @Adithyaphani , I'm on this, sorry for the delay! |
|
Okay @oscar-wallis |
|
Hi @Adithyaphani , could you check over our new AI Policy and confirm you have read and understood it? In line with the new AI Policy, please also disclose whether or not you used AI in this PR and if so, which? A similar message is being sent to all open PRs. |
|
@oscar-wallis the usage of AI was only up to helping few nitty-gritty points of issue and PR description part. But the logic and code part is done by myself. Looking forward to your response and happy to make further changes if needed. |
Coverage Report for CI Build 31481790807Coverage decreased (-0.03%) to 89.764%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions1 previously-covered line in 1 file lost coverage.
Coverage Stats
💛 - Coveralls |
Signed-off-by: AdithyaPhani <adithyaphani7@gmail.com>
|
@oscar-wallis Pushed This PR removes the qubit auto-alignment code from Note: |
|
@oscar-wallis Looking forward to your review and happy to make further changes if needed. |
|
@oscar-wallis looking forward to your response and happy to make further changes if needed. |
|
@oscar-wallis looking forward to your response and happy to make further changes if needed. Are you further looking to move forward with this issue or not? |
|
@oscar-wallis and @OkuyanBoga and @garrison looking forward to your review on this PR. Would like to make further changes if needed. |
###Summary:
Issue #1024 fix
Removes the automatic qubit padding behavior from the helper utilities and cleans up deprecated logic related to qubit derivation.
This PR updates the helper behavior to make qubit handling stricter and more explicit by avoiding silent circuit modifications.
###Details and comments:
Removed the automatic qubit padding mechanism from the helper:
Deleted deprecated logic from derive_num_qubits
Updated the helper to raise an explicit error instead of silently padding circuits
Improves transparency and prevents unexpected circuit behavior during execution
Simplifies the internal qubit-handling flow and removes deprecated pathways This change may affect workflows that previously relied on implicit circuit padding behavior.