-
Notifications
You must be signed in to change notification settings - Fork 569
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
Update DecompositionUndefinedError
for Exp
class
#4571
Conversation
Hello. You may have forgotten to update the changelog!
|
[sc-44544] |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4571 +/- ##
=======================================
Coverage 99.64% 99.64%
=======================================
Files 376 376
Lines 33071 33076 +5
=======================================
+ Hits 32955 32960 +5
Misses 116 116
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 🚀
**Context:** When calling the decomposition for the `Exp` class with `num_steps` specified, an error was raised suggesting to set `num_steps`. The real source of the error was caused because the underlying operator was not unitary. The error message was misleading. **Description of the Change:** Update the decomposition function to perform added validation before raising error. **Benefits:** More informative error messages. **Possible Drawbacks:** None **Related GitHub Issues:** Fixes #4537
Context:
When calling the decomposition for the
Exp
class withnum_steps
specified, an error was raised suggesting to setnum_steps
. The real source of the error was caused because the underlying operator was not unitary. The error message was misleading.Description of the Change:
Update the decomposition function to perform added validation before raising error.
Benefits:
More informative error messages.
Possible Drawbacks:
None
Related GitHub Issues:
Fixes #4537