fermisurfaces / IFermi

Fermi surface generation, analysis and visualisation.
https://fermisurfaces.github.io/IFermi/
MIT License
87 stars 33 forks source link

BUG: Too many values to unpack #344

Closed KYZeng111 closed 1 year ago

KYZeng111 commented 1 year ago

I just installed ifermi using anaconda in windows wsl, and try to run "ifermi info" in example/MgB2 folder. The error also appears if I run pytest.

image

paulneves77 commented 1 year ago

It seems that trimesh added an extra output to slice_faces_plane in August (see https://github.com/mikedh/trimesh/commit/63c60951c20df46d722cc150dc35b66d157e3866)

This error can be fixed by assigning the additional output to a dummy output in line 743 in surface.py: vertices, faces, _ = slice_faces_plane(vertices, faces, -normal, center)

I have submitted a pull request.

utf commented 1 year ago

This is now fixed in version 0.3.2. Thanks for your help @paulneves77