search

dimchris / mdanalysis

Automatically exported from code.google.com/p/mdanalysis
0 stars 0 forks source link

UTF-8 encoded filenames are not supported #88

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Let's say we have a file named "test.pdb" in the current directory
2. Run the following command:
python -c "import MDAnalysis; test = MDAnalysis.Universe(u'test.pdb'); print 
test;"

What is the expected output? What do you see instead?
expected: something like <Universe with 12345 atoms>
instead: RuntimeError due to recursion limit

Original issue reported on code.google.com by sebastie...@gmail.com on 25 Nov 2011 at 10:41

GoogleCodeExporter commented 9 years ago 
Is this with the svn trunk?

I thought I fixed this (or something similar) in r917. At least the following 
works for me:

  >>> import MDAnalysis
  >>> from MDAnalysis.tests.datafiles import *
  >>> u = MDAnalysis.Universe(unicode(PSF), unicode(DCD))
  >>> print u
  <Universe with 3341 atoms>
  >>> print unicode(PSF)
  >>> unicode(PSF)
  u'/Users/oliver/.local/lib/python2.6/site-packages/MDAnalysisTestData-0.7.4-py2.6.egg/MDAnalysisTestData/data/adk.psf'

Original comment by orbeckst on 27 Nov 2011 at 4:59

GoogleCodeExporter commented 9 years ago 
For completeness sake: With r919

 python -c "import MDAnalysis; test = MDAnalysis.Universe(u'/Users/oliver/.local/lib/python2.6/site-packages/MDAnalysisTestData-0.7.4-py2.6.egg/MDAnalysisTestData/data/adk.psf'); print test;"
<Universe with 3341 atoms>

... "works for me" --- does r919 still fail for you?

Original comment by orbeckst on 27 Nov 2011 at 5:11

GoogleCodeExporter commented 9 years ago 
  python -c "import MDAnalysis; test = MDAnalysis.Universe(u'/Users/oliver/.local/lib/python2.6/site-packages/MDAnalysisTestData-0.7.4-py2.6.egg/MDAnalysisTestData/data/adk.psf', u'/Users/oliver/.local/lib/python2.6/site-packages/MDAnalysisTestData-0.7.4-py2.6.egg/MDAnalysisTestData/data/adk_dims.dcd'); print test;"
  <Universe with 3341 atoms>

(Need to supply a trajectory to trigger the bug if it's still there --- but 
works for me in r919.)

Original comment by orbeckst on 27 Nov 2011 at 4:21

GoogleCodeExporter commented 9 years ago 
Note to self, always check svn logs before filing an issue: you did fix this in 
r917.
I hit that bug with the 0.7.4 release (used to build the debian packages) not 
with the dev version.
This bug is then present in the 0.7.4 release but won't in the next release.

Status changed to Verified.

Original comment by sebastie...@gmail.com on 28 Nov 2011 at 3:34