Skip to content
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

Raising QiskitError #30

Closed
woodsp-ibm opened this issue Jul 31, 2023 · 0 comments · Fixed by #41
Closed

Raising QiskitError #30

woodsp-ibm opened this issue Jul 31, 2023 · 0 comments · Fixed by #41

Comments

@woodsp-ibm
Copy link
Member

While part of Qiskit, qiskit.algorithms raised, in a few places, QiskitError directly. Now there is an AlgorithmError (similar to the apps having their own exceptions) which is also used - maybe this should be raised now in those places too now that algorithms are no longer part of Qiskit. I will note AlgorithmError, and the apps' exceptions, derive from QiskitError, but it will list as the former type in a stacketrace which maybe is more desirable than directly raising QiskitError which may cause confusion now that algorithms is no longer part of Qiskit.

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 a pull request may close this issue.

1 participant