Opened 6 years ago
Closed 6 years ago
#17894 closed enhancement (fixed)
JSMol in the IPython notebook
Reported by: | vbraun | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-6.6 |
Component: | notebook | Keywords: | days64 |
Cc: | Merged in: | ||
Authors: | Volker Braun | Reviewers: | Eric Gourgoulhon |
Report Upstream: | N/A | Work issues: | |
Branch: | 3fce004 (Commits) | Commit: | 3fce004fe8a539d2998fff6abcc9b6c14baedd38 |
Dependencies: | #17826 #17234 | Stopgaps: |
Description (last modified by )
Change History (14)
comment:1 Changed 6 years ago by
- Component changed from PLEASE CHANGE to notebook
- Dependencies set to #17826 #17234
- Type changed from PLEASE CHANGE to enhancement
comment:2 Changed 6 years ago by
- Branch set to u/vbraun/jsmol_in_the_ipython_notebook
comment:3 Changed 6 years ago by
- Commit set to b78260d046fff40e1b672dfc14e680cc3442d61f
- Description modified (diff)
comment:4 Changed 6 years ago by
- Status changed from new to needs_review
comment:5 Changed 6 years ago by
That's nice!
However, when I click on the right button to get the jsmol menu, I cannot access to submenus (e.g. zoom). This is with Sage 6.6.rc0.
Also, there was a small conflict in src/sage/repl/ipython_kernel/install.py when merging into Sage 6.6.rc0 (instance.install_spec()
--> instance._install_spec()
and instance.symlink_resources()
--> instance._symlink_resources()
)
comment:6 Changed 6 years ago by
- Commit changed from b78260d046fff40e1b672dfc14e680cc3442d61f to f1968b9e9b367ad40ef9139b46ae4d40900d3ff3
Branch pushed to git repo; I updated commit sha1. New commits:
f1968b9 | Merge tag Sage-6.6.rc0 into #17894
|
comment:7 Changed 6 years ago by
I don't think the menu is possible. In the current ipython notebook, everything must be embedded in the cell output. But jsmol has no interface to pass the menu in, it insists on loading it in a separate request.
comment:8 Changed 6 years ago by
- Keywords days64 added
- Reviewers set to Eric Gourgoulhon
- Status changed from needs_review to positive_review
OK I see. Anyway, it is already very nice to have interactive 3D plots in the ipython notebook. Thank you for this work!
comment:9 Changed 6 years ago by
Wish I had noticed earlier you had another instance of os.environ[IPYTHONDIR]
in there before it was positively reviewed. Expect a follow up ticket normalizing it to get_ipython_dir
.
comment:10 Changed 6 years ago by
thanks for noticing, I'll add it here
comment:11 Changed 6 years ago by
- Commit changed from f1968b9e9b367ad40ef9139b46ae4d40900d3ff3 to 3fce004fe8a539d2998fff6abcc9b6c14baedd38
- Status changed from positive_review to needs_review
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:
3fce004 | use get_ipython_dir
|
comment:12 Changed 6 years ago by
- Status changed from needs_review to positive_review
comment:13 Changed 6 years ago by
thanks.
comment:14 Changed 6 years ago by
- Branch changed from u/vbraun/jsmol_in_the_ipython_notebook to 3fce004fe8a539d2998fff6abcc9b6c14baedd38
- Resolution set to fixed
- Status changed from positive_review to closed
Last 10 new commits:
Merge #17234: Rich output and the IPython Notebook
Merge in Sage 6.6.beta2
Merge #17288: MathJax spkg for ipython notebook
Use the new IPython kernel stuff for Sage
delete the old ipython notebook templates
Use local docs, icon
Merge #17234: Rich output and the IPython Notebook
Merge Trac #17897: Jsonschema package
Merge in Sage 6.6.beta3
Enable JSmol in the IPython notebook