Skip to content
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

Incorrect jmol rendering when EMBEDDED_MODE is not set #10323

Closed
pcpa mannequin opened this issue Nov 24, 2010 · 3 comments
Closed

Incorrect jmol rendering when EMBEDDED_MODE is not set #10323

pcpa mannequin opened this issue Nov 24, 2010 · 3 comments

Comments

@pcpa
Copy link
Mannequin

pcpa mannequin commented Nov 24, 2010

I made a post to @save-devel and was advised by a jmol developer to open a trac ticket, so, here it is.

Standard notebook output looks like http://img269.imageshack.us/img269/7279/notebookandtutorial.png but when run in the command line, the output looks like http://img831.imageshack.us/img831/3907/jmolwindow.png

The solution I used was to ignore EMBEDDED_MODE and use the same logic as when generating notebook data.

Note that the patch also does not use sage-native-execute and also pass "-s" to jmol, but these are issues with the mandriva rpm package, where it uses the system wide jmol installation, and workarounds an apparent issue with newer jmol that will not understand the argument is a script if not using the "-s" option.

Upstream: Reported upstream. Developers acknowledge bug.

CC: @mkoeppe @slel @DaveWitteMorris

Component: user interface

Keywords: jmol

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/10323

@pcpa pcpa mannequin added c: user interface labels Nov 24, 2010
@pcpa pcpa mannequin assigned williamstein Nov 24, 2010
@pcpa
Copy link
Mannequin Author

pcpa mannequin commented Nov 24, 2010

jmol related sage-4.6 mandriva rpm patch

@fchapoton
Copy link
Contributor

comment:1

Attachment: jmol.patch.gz

very old, no longer relevant. Can we close as invalid ?

@mkoeppe
Copy link
Member

mkoeppe commented Aug 11, 2021

Reviewer: Matthias Koeppe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants