#12299: Advance Jmol Interactive Features in Flask Notebook
------------------------------------------------------------+---------------
       Reporter:  gutow                                     |         Owner:  
jason, mpatel, was                                        
           Type:  enhancement                               |        Status:  
needs_review                                              
       Priority:  major                                     |     Milestone:  
sage-5.2                                                  
      Component:  notebook                                  |    Resolution:    
                                                        
       Keywords:  Jmol, 3D, notebook                        |   Work issues:    
                                                        
Report Upstream:  Workaround found; Bug reported upstream.  |     Reviewers:  
Karl-Dieter Crisman, Steven Trogdon, Punarbasu Purkayastha
        Authors:  Jonathan Gutow                            |     Merged in:    
                                                        
   Dependencies:  #11080,#11078,#11503                      |      Stopgaps:    
                                                        
------------------------------------------------------------+---------------

Comment (by gutow):

 Replying to [comment:100 kcrisman]:
 > Okay, I've created a reviewer patch.  Now I know the formatting is right
 because I added this to the reference manual and looked.

 > So in theory, this should be wonderful.  Only now I've horribly broken
 something, because these files in the {{{os.path.exists}}} tests no longer
 exist and those tests fail!  I have precisely zero explanation; every time
 I made them live in the current directory, like just {{{DNA.png}}}, it
 worked, every time I put them someplace else it didn't.
 >
 Your problem is in line 163.  I ran into this when I was writing the test
 code.  You have too many layers of escaping on the endofline.  This is why
 I used the form:
 {{{
 sage: script = 'set defaultdirectory "'+archive_name+'"\n script SCRIPT\n'
 }}}
 rather than the
 {{{
 sage: script = 'set defaultdirectory "%s"\\n script
 SCRIPT\\n'%archive_name
 }}}
 you used. Alternatively, your code works if you change it to:
 {{{
 sage: script = 'set defaultdirectory "%s"\n script SCRIPT\n'%archive_name
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12299#comment:103>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to