-
Notifications
You must be signed in to change notification settings - Fork 60
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
Deprecate Array class #343
Deprecate Array class #343
Conversation
…e module level documentation for array and dispatch
|
||
The ``array`` and ``dispatch`` submodules of Qiskit Dynamics have been deprecated as of version | ||
0.5.0. The use of the ``Array`` class is no longer required to work with different array | ||
libraries in Qiskit Dynamics, and is broken in some cases. Refer to the user guide entry on |
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.
Is it possible add any link to this userguide?
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.
Yes thanks, good idea. I've just added the link to this text.
Also this comment made me realize I accidentally put this warning in the arraylias
submodule docs, as opposed to the dispatch
submodule docs :D. I've also corrected this.
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.
Thanks. LGTM!
Summary
Deprecating
Array
class, and adding release upgrade note.Details and comments