search

jjtobin / auto_selfcal

MIT License
17 stars 8 forks source link

parallel cannot be defined if mpicasa is not installed #36

Closed SebaMarino closed 8 months ago

SebaMarino commented 8 months ago

I'm trying out this for the first time on OSX mpicasa, so I tried casa -c auto_selfcal.py, which triggered some errors such as

from casampi.MPIEnvironment import MPIEnvironment ModuleNotFoundError: No module named 'casampi'

I had to manually comment the lines using casampia and define parallel=False. Perhaps this is something that could be improved in the future with a try/except statement.

jjtobin commented 8 months ago

Hi Seba,

Thanks for your feedback, indeed we've only been testing on Linux systems. I had overlooked the fact that Mac builds of CASA do not include MPI capability which resulted in your issue. I will put in an update with a try/except as you suggest.

jjtobin commented 8 months ago

I just added a commit with the try/except if you want to test it on Mac.

SebaMarino commented 8 months ago

Hi John, thanks for the quick reply. I think run_selfcal.py also requires the same fix.

I also noticed that the readme file should also say that run_selfcal.py has to be copied to the working directory.

jjtobin commented 8 months ago

Thanks for pointing that out regarding run_selfcal.py!