BUG: ValueError in test_magnetic_orderings

[janosh]

@tpurcell90 i skipped the test_magnetic_orderings you added in #922 in this commit e7ac025. it started failing when upgrading to the latest pymatgen. i couldn't figure out why it started failing. if you have time, could you take a look?

[tpurcell90]

Yes I can do this today

[tpurcell90]

I found the problem, it is in pymatgen. I will create a PR there and discuss with @ansobolev about the change.

[tpurcell90]

@janosh The pymatgen fix was just merged in so this should now work

[janosh]

thanks for letting me know. we're still waiting on a new release, right?

[tpurcell90]

Ya, I am not sure when the next pymatgen release will be, but it should work with the new one

[JaGeo]

@tpurcell90 You can always ask nicely on the repo for a release. Usually, the release follows quite fast. 🙂

[tpurcell90]

Okay I asked about a new version in the PR for this fixed