pyiron / pyiron_potentialfit

Fitting machine learning interatomic potentials with pyiron
http://potentials.rub.de
BSD 3-Clause "New" or "Revised" License
3 stars 1 forks source link

ModuleNotFoundError: No module named 'pyiron_potentialfit.spgfit' #107

Closed lfzhu-phys closed 3 months ago

lfzhu-phys commented 3 months ago

The module pyiron_potentialfit.spgfit cannot be found.

The input is as following:

from pyiron_potentialfit.spgfit import structures as pstruct
re(pstruct)

The error message is as following:

--------------------------------------------------------------------
/cmmc/ptmp/pyironhb/mambaforge/envs/pyiron_mpie_cmti_2024-07-01/lib/python3.11/site-packages/h5py/__init__.py:36: UserWarning: h5py is running against HDF5 1.14.3 when it was built against 1.14.2, this may cause problems
  _warn(("h5py is running against HDF5 {0} when it was built against {1}, "

---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[2], line 1
----> 1 from pyiron_potentialfit.spgfit import structures as pstruct
      2 re(pstruct)

ModuleNotFoundError: No module named 'pyiron_potentialfit.spgfit'
------------------------------------------------------------------------------
pmrv commented 3 months ago

In the new environments from @niklassiemer pyiron_potentialfit was still on an older version. I've updated the version and it should be available again in the environment released next week. For now you should be able to go back to the pyiron/old_latest environment.

niklassiemer commented 3 months ago

Thanks

niklassiemer commented 3 months ago

@pmrv is there actually a test possible on the docker-stacks to ensure the environment is working?

pmrv commented 3 months ago

@pmrv is there actually a test possible on the docker-stacks to ensure the environment is working?

I don't know, what would I need to check?

pmrv commented 3 months ago

@niklassiemer The testing workflow is passing here, is that what you mean?

lfzhu-phys commented 3 months ago

@pmrv Thanks a lot.

niklassiemer commented 3 months ago

I meant that the old version had been inkompatibel, which I would like to catch :)