-
Notifications
You must be signed in to change notification settings - Fork 41
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
Migrate and fix Optimal control tutorials from qutip-notebooks to qutip-tutorials #122
base: main
Are you sure you want to change the base?
Conversation
I checked the |
@ajgpitch I have fixed this and added overview. I think we should figure out, which order we want these notebooks to appear. I can name them in that particular order to show in the website index. |
Definitely looking better now. Still focusing on the Hadamard example...
|
I have removed the dependence on These are the versions I am using. |
@ajgpitch I am not able to understand this error
Rest I think everything is fixed |
Maybe it is the empty lines in cell 1 and also perhaps |
Suggest renames:
btw, it would be nice add state_to_state to the Hadamard example, perhaps first, showing transfer from |1> to |+> (or |0> to |+> , whichever one is the Hadamard, I can never remember), then show that Hadamard does the same. |
@ajgpitch I have updated the names. I am thinking for making changes to the content, I can open other PR to integrate this nice idea and maybe more. |
Tests failing because links need updating in |
Migrate and update the Optimal Control tutorials from the
qutip-notebooks
repository to thequtip-tutorials
repository. Ensure all tutorials are compatible withqutip-qtrl
instead of the olderqutip.control
module. This involves revising code, validating outputs, and updating instructions to reflect changes functionality. It also removes thelogging
to make the tutorials more compact.To do: