-
Notifications
You must be signed in to change notification settings - Fork 31
Add try-qutip link and description. #170
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
Conversation
AGaliciaMartinez
left a comment
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.
I read the html document and really nice work. Very well explained! I would say it is ready to be merged.
I was looking to compile the webpage locally to review it formatted but I do not seem to find any documentation for it. Do we have it somewhere by any chance? I am not very familiar with working with webpages.
|
Jake gave a nice explanation of how to view this on your own GitHub pages -- #67 (comment). One can also build locally by installing and running Jekyll. |
|
Thanks for taking a look, Asier! |
nathanshammah
left a comment
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 @hodgestar.
- It could be nice to land to the summary index.ipynb notebook that we (once) had when clicking on Try qutip. If there is a way to do it, it would be great.
- When I tried to run a notebook, it did not work
|
I should add qutip_qip to what's installed! Doh. :) |
|
Thank you for looking Nathan. It would be nice to land on a summary notebook. Will try add. |
|
I updated try-qutip to QuTiP 5.1.1 and now the quantum circuits render beautifully. @Ericgig @ajgpitch @nwlambert @pmenczel -- would some of you mind giving it another whirl just to make sure it works for people other than me? I'd like to merge these even if there are some outstanding issues & then keep track of the known issues in the GitHub issues for the repository. |
|
Seems to be working pretty well, and I think it is really cool and useful! Some comments:
|
|
Thanks for taking a look @pmenczel.
Yay!
Building and deploying is completely automated in a GitHub workflow, so pushing a commit to master or pressing the right button in GitHub Actions will do everything. The tricky part is that the whole jupyterlite and emscripten-forge ecosystem is still evolving rapidly, so there are regularly big changes that need to be figured out. E.g. Currently we need to deploy from a dev emscripten-forge repository (see https://github.com/qutip/try-qutip/blob/5e2f773bff768d4b425134c18e4868159bfb9773/lite/try-qutip-env.yaml#L4).
Let me add ipywidgets to what's installed and see what happens.
It would be nice to create a landing page. Not quite sure how to do that yet. |
|
I added |
|
It works in general, but there are quite a few errors and warnings: The animation in block-sphere-animation fails: And there is: Any plan for automated action running all notebook in this environment? |
… working in try-qutip.
|
@Ericgig Thank you for taking a look:
The error
This is an interesting idea, but it automating running Firefox or Chrome headless in GitHub CI with all of the browser control needed to launch notebooks and read results from them sounds painful. None of these seem like show stoppers to me. |
|
@pmenczel I'll try to find some time to generate an index by borrowing some of the logic from the tutorials generation. |
|
I added a script for generating a Markdown index for the tutorials to qutip/qutip-tutorials#90. Once its reviewed and merged, I can use if from the try-qutip site. |
|
The script has been merged and the generated README.md now appears on https://qutip.org/try-qutip/. I'm not year sure how to get an .md file to be displayed by the Markdown Preview by default though, so currently one has to right-click on the file and select There is an extension that allows opening URLs from the web in Jupyterlite by appending |
|
The The URL after |
pmenczel
left a comment
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.
Thank you, the fromURL is useful and it's good to have the readme!
Sorry for not being very clear earlier, my user experience complaints were mostly about that "launcher". Most of the content in there is useless and/or lacks clear description, and it would be good to have a link to the notebooks or the readme there as well.
However, that is something that could be changed later, if it is possible to be changed at all.
|
@pmenczel The launcher can be configured. I created an issue for doing so at qutip/try-qutip#6 but I'll leave doing the work for later. Any concrete suggestions about what should be in the launcher welcomed. |

I've moved the try-qutip repository to https://github.com/qutip/try-qutip/ and it's now available at https://qutip.org/try-qutip/.
This PR adds a description of what try-qutip is and links to that description from the navigation bar.