Skip to content

Conversation

jan-janssen
Copy link
Member

No description provided.

@jan-janssen
Copy link
Member Author

@srmnitc This should fix the error of pympipool or pylammpsmpi just being stuck without any error message. Can you test it with typically LAMMPS error messages in calphy?

@srmnitc
Copy link
Member

srmnitc commented Sep 15, 2023

I get the error

  File "/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/calphy/lattice.py", line 26, in <module>
    from pylammpsmpi import LammpsLibrary
  File "/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/pylammpsmpi/__init__.py", line 1, in <module>
    from pylammpsmpi.wrapper.extended import LammpsLibrary
  File "/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/pylammpsmpi/wrapper/extended.py", line 5, in <module>
    from pylammpsmpi.wrapper.base import LammpsConcurrent
  File "/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/pylammpsmpi/wrapper/base.py", line 5, in <module>
    from pylammpsmpi.wrapper.concurrent import LammpsConcurrent
  File "/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/pylammpsmpi/wrapper/concurrent.py", line 9, in <module>
    from pympipool import RaisingThread, cancel_items_in_queue, interface_bootup
ImportError: cannot import name 'RaisingThread' from 'pympipool' (/home/menon/miniconda3/envs/calphy/lib/python3.10/site-packages/pympipool/__init__.py)

@jan-janssen
Copy link
Member Author

jan-janssen commented Sep 15, 2023

@srmnitc Can you change the import to from pympipool.shared import RaisingThread?

Ah I realised I already did this in pyiron/pylammpsmpi#138

@jan-janssen jan-janssen merged commit 4336e78 into main Sep 15, 2023
@jan-janssen jan-janssen deleted the more_verbose branch September 15, 2023 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants