-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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 |
@ajgpitch After changing the links, now it looks correct to me. But the tests still seem to fail. |
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: