TypeError: list indices must be integers or slices, not tuple

[jnluy]

The test failed on a new installation of effmass on Python 3.6.

[lucydot]

Hello, thanks for flagging this up - can you give me a couple of more details so I can trace the problem - which test failed?

[jnluy]

Hi. I made a fresh install of python 3.6.8. And then followed the instructions in the readme. pip3 install --user effmass python3 -m pytest The error log is attached. The same happens for python3.7. Am Do., 16. Mai 2019 um 16:11 Uhr schrieb Lucy Whalley < notifications@github.com>:

…

Hello, thanks for flagging this up - can you give me a couple of more details so I can trace the problem - which test failed? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#14?email_source=notifications&email_token=AMC3JMWQBIBIZBEK25A6QMDPVVTPHA5CNFSM4HNMJOX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVR5Z6Q#issuecomment-493083898>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMC3JMUGKXHR6ROXFUTKPVLPVVTPHANCNFSM4HNMJOXQ> .

[lucydot]

Hello, I'm afraid the error log isn't attached - I think you will need to log into github and drag and drop the file (https://help.github.com/en/articles/file-attachments-on-issues-and-pull-requests). Automated testing of effmass happens here, and it seems to be passing all tests, so I'm a little stumped why it is failing for you - might be something system dependant. Just to check that you are running the python3 -m pytest command from inside the effmass folder? (pip3 show effmass will tell you where effmass is installed)

[jnluy]

err.log

[bjmorgan]

This looks like it might be an issue introduced by a recent change to the vasppy API, which is used to parse the PROCAR files. If so, it should be fixed in this pull request: #15

[lucydot]

Hi, thanks @bjmorgan for submitting the PR request - this has been merged with effmass, and I have made a new release (v1.1.0) which should now be available through PyPI. @jnluy if you update effmass via pip it should pass all tests 🤞

[lucydot]

Once you confirm the problem is fixed @jnluy I'll close the issue - thanks again for flagging it up.
-Lucy

[jnluy]

pytest still fails, but with another error now
err2.log

[bjmorgan]

pytest still fails, but with another error now
err2.log

The error log appears to show the old version of effmass. Can you please check what version you have installed using pip list?

[jnluy]

pytest still fails, but with another error now
err2.log

The error log appears to show the old version of effmass. Can you please check what version you have installed using pip list?

You are right. I was sitting in the wrong folder when calling pytest. Thank you for the quick fix!

[lucydot]

Thanks to PR #15 from @bjmorgan the latest versions of vasppy and effmass are now compatible with each other and so I am closing this issue.