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

openpyxl and psychopy/data.py #1314

Closed
terronymous-boosh opened this issue Nov 25, 2016 · 6 comments
Closed

openpyxl and psychopy/data.py #1314

terronymous-boosh opened this issue Nov 25, 2016 · 6 comments
Labels
🐞 bug Issue describes a bug (crash or error) or undefined behavior.

Comments

@terronymous-boosh
Copy link

Just ran a manual install of 1.84.2, loaded all the pre-requisites and ran into the following error:

trialList=data.importConditions(u'SequentialPriming.xlsx'),
  File "C:\Python27\lib\site-packages\psychopy\data.py", line 2668, in importConditions
    raise ImportError('openpyxl is required for loading excel '
ImportError: openpyxl is required for loading excel format files, but it was not found.

it seems like as of godknowswhen openpyxl (version 2.4.1 on my machine) no longer has the method cell.get_column_letter, so instead you would need to change line 29 in data.py from

from openpyxl.cell import get_column_letter

to

from openpyxl.utils import get_column_letter

@hoechenberger
Copy link
Contributor

hoechenberger commented Nov 25, 2016

Could you try with latest master? I believe this was fixed in #1276
Could be a dupe of #1274 then.

@hoechenberger hoechenberger added 🐞 bug Issue describes a bug (crash or error) or undefined behavior. library ⏰ waiting for feedback labels Nov 25, 2016
@terronymous-boosh
Copy link
Author

terronymous-boosh commented Nov 25, 2016 via email

@hoechenberger
Copy link
Contributor

hoechenberger commented Nov 25, 2016

pip install --upgrade git+https://github.com/psychopy/psychopy

should do the trick. If you don't have git installed, you can try

pip install --upgrade https://github.com/psychopy/psychopy/archive/master.zip

@hoechenberger
Copy link
Contributor

@terronymous-boosh, any news on this one?

@Soraltan
Copy link

Soraltan commented Jan 18, 2017

Hi, I had exactly the same error.

The suggested upgrade from the git reps did the trick. It is solved, as far as I can see.

@peircej
Copy link
Member

peircej commented Jan 18, 2017

Thanks for confirming this is fixed in repos. Will be released very soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Issue describes a bug (crash or error) or undefined behavior.
Projects
None yet
Development

No branches or pull requests

4 participants