-
-
Notifications
You must be signed in to change notification settings - Fork 480
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
jmol on commandline broken #9232
Comments
comment:1
Does it work if you install the new version of Jmol? See trac # 9238. I think this broke when the notebook was put inside an egg. I took that into account with the new version of Jmol. |
comment:2
I installed http://www.uwosh.edu/faculty_staff/gutow/Jmol_for_SageNoteBook-1.1.spkg (#9238) but it did not change anything. The verbose output above indicates that Jmol.jar is never opened, so it is not particularly surprising that an updated Jmol spkg has no effect. |
comment:3
It was worth a try as that still could have been a hidden Jmol.jar file not found error. This is different than the problem I addressed. Sorry. I would guess it is a problem with the "sphere" command. Does it work with a simple plot3d command? I've tested the 1.1 version on MacOS and Ubuntu and it seems to work for me.
|
comment:4
I only used the
|
comment:5
Try installing the Jmol for notebook again. It looks like the file on my server got corrupted and was missing the Jmol.jar application file as well as some others..Not sure what happened but we've had some power outages and the server may have been rebuilt from backups. I've uploaded a new copy. Because the name hasn't changed you will probably have to do some surgery to get it to download and do an install. You will need to delete any copies of the .spkg (or its uncompressed directory) that you find in any of these subdirectories of your sage install before trying an install. If you don't get a message about downloading the file you missed something: Let us know if this fixes it. Sorry for the inconvenience.
|
comment:6
I did a
and it re-downloaded the Jmol spkg. Still fails with the same error message. |
comment:7
There is something special about your system...I've just retested that things work for Sage 4.4.3 on MacOS and Ubuntu Linux. Let's see if everything actually got installed: If everything is there, I'm mystified. If everything is not, then something is wrong with the install for your system and the .spkg needs to be fixed. Jonathan
|
comment:8
I saved the
But sage install zlib in
Note: From within Sage, on the other hand,
|
comment:9
For the record, creating a symlink $SAGE_ROOT/local/lib64 pointing to $SAGE_ROOT/local/lib works. Another workaround is adding LIBRARY_PATH to DYLD_LIBRARY_PATH in sage-native-execute (from logix/#sage-devel). I still don't understand the underlying cause, where does python get the incorrect path from? |
Attachment: trac_9232_plot3d_base_pyx.patch.gz plot3d/base.pyx fixes to launch Jmol from cmd line |
Notebook fixes for launch Jmol from cmd line |
Reviewer: Jason Grout, Karl-Dieter Crisman |
comment:13
Attachment: trac_9232_notebook_fixes.patch.gz The above patches combined with installation of a newer version of Jmol fix this problem for Sage 4.6.2. To install the new Jmol
Note that my linux and MacOS versions of sage seemed to have slightly different mercurial tracking. On MacOS you get a warning about the SageMenu.mnu file, when you apply the notebook_fixes patch. That is OK, the other untracked files are properly added. |
Author: Jonathan Gutwo |
comment:14
This will have to be part of a new SageNB package before it can be merged. See also #9238. |
Changed author from Jonathan Gutwo to Jonathan Gutow |
comment:24
Replying to @vbraun:
Thanks for that feedback very much. Ok, then I propose that that patch gets moved here, and then #10286 can be just about sage-native-execute. Am I correct in assuming that only that patch from #10286 is needed to fix the behavior? |
comment:25
Yes, it would be fine to move |
Attachment: trac_9232_call_jmol_correctly.patch.gz Apply this second. |
Changed reviewer from Jason Grout, Karl-Dieter Crisman to Jason Grout, Karl-Dieter Crisman, Volker Braun |
Changed author from Jonathan Gutow to Jonathan Gutow, Volker Braun |
This comment has been minimized.
This comment has been minimized.
comment:26
Okay, this should be all okay then. |
comment:27
Things are a little bit of a mess because of the potential for flask changeover in SageNB. What do people think of this?
|
comment:28
The flask notebook has already split off of the current notebook. So any patches that are applied to the current sage notebook will also need to be applied to the new notebook, probably in parallel. I'm already working on moving #9238 over to the new notebook. It's just a matter of applying the patch to the new notebook, as far as I can tell. (We'll find out better once we actually get jmol working with the new notebook!) |
comment:29
Should I continue to put my patches into #9238? I've still got some fixes that should reduce memory problems making Safari somewhat more reliable and I may be able to get the advanced controls working in IE. |
This comment has been minimized.
This comment has been minimized.
comment:30
Jonathan, definitely keep working on #9238. Jason can just put those things in the new notebook. Having something working completely is a very good idea, and we can continue to test it on the old notebook, which is still the current notebook, after all. I do think we need to also add the piece of #9238 that changes the lighting. Currently I have good lighting in the command line and bad lighting in the notebook, even without the new jmol - someone the new jmol is called from the notebook, even though I didn't install it in this version!!! I even checked this in the advanced menu. Mystifying. I'm just testing quick, and then I'll post patches. Since they aren't mine, I feel free to give positive review. |
Attachment: trac_9232-lighting.patch.gz |
comment:31
Apply attachment: trac_9232_call_jmol_correctly.patch and attachment: trac_9232-lighting.patch. |
This comment has been minimized.
This comment has been minimized.
comment:32
Just a ping to let the release manager know this is ready for alpha4. :-) |
comment:33
Please confirm that the |
comment:34
I meant to say more precisely: please confirm that the patch attachment: trac_9232_notebook_fixes.patch may be ignored. |
comment:35
That is correct. Only the two patches listed at the end of the description are necessary. The other two have been incorporated into ticket #9238 because they are enhancements to how Jmol works. |
Merged: sage-4.7.alpha4 |
I can't see Graphics3d objects on the command line. For example,
The command returns without starting jmol or producing any other graphical output.
Strangely enough, I can start jmol from the sage command line:
The following also works and shows a tachyon-generated plot:
More verbosity:
For the record, I'm running Fedora 13 x86_64
Apply attachment: trac_9232_call_jmol_correctly.patch and attachment: trac_9232-lighting.patch.
CC: @kcrisman @jasongrout @jdemeyer
Component: graphics
Author: Jonathan Gutow, Volker Braun
Reviewer: Jason Grout, Karl-Dieter Crisman, Volker Braun
Merged: sage-4.7.alpha4
Issue created by migration from https://trac.sagemath.org/ticket/9232
The text was updated successfully, but these errors were encountered: