#9238: J. Gutow's update to Jmol in the notebook...
------------------------------------------------+---------------------------
   Reporter:  gutow                             |       Owner:  gutow       
       Type:  enhancement                       |      Status:  needs_review
   Priority:  major                             |   Milestone:  sage-4.7    
  Component:  notebook                          |    Keywords:              
     Author:  Jonathan Gutow                    |    Upstream:  N/A         
   Reviewer:  Jason Grout, Karl-Dieter Crisman  |      Merged:              
Work_issues:                                    |  
------------------------------------------------+---------------------------

Comment(by kcrisman):

 Replying to [comment:86 jason]:
 > Hmmm...it looks like those things should be using os.path.join, like
 this:
 > {{{
 > sage: os.path.join('test/','asdf')
 > 'test/asdf'
 > sage: os.path.join('test','asdf')
 > 'test/asdf'
 > }}}
 Yeah, we had this problem with some of the temp files in e.g. R interface
 before.

 As to the Python temp files, this is the system-wide `SAGE_TMP`, so you'd
 have to ask whoever wrote that why they didn't use builtin Python stuff -
 probably William or Tom or someone in prehistory...

 As to Jonathan's question, I think this can just be fixed here for now,
 with a followup ticket for fixing it anyplace else it might cause
 problems.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9238#comment:89>
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