arhstevens / DarkSage

The Dark Sage semi-analytic model
MIT License
11 stars 3 forks source link

Enabling MPI #3

Open manodeep opened 6 years ago

manodeep commented 6 years ago

DarkSage currently doesn't have MPI properly enabled and does not run on multiple nodes on ozstar. Porting over the sage style of MPI parallelisation should be fairly straightforward.

@arhstevens Is there any reason that the sage-style MPI feature was removed from DarkSage?

arhstevens commented 6 years ago

I don't think it ever was "removed" (can't see anything obvious that would suggest it was after browsing the commit history). I think it was simply a case of Dark Sage branching off SAGE before MPI was properly implemented for SAGE. Getting this implemented properly has been on my to-do list. Hasn't been a priority because it hasn't been a problem thus far. Sounds like it is time to get it done!

manodeep commented 4 years ago

@arhstevens Looking at the source, the MPI seems to have been implemented properly. Is that not the case?

arhstevens commented 4 years ago

Give it a go and find out, I guess. I never got it to work with the actual MPI flag on. Has been quite a while since I tried though.

manodeep commented 4 years ago

From my read, the MPI is fully functional. You can also remove the calls to time and sleep from main.c. Should work just fine in MPI.

If the MPI mode still doesn't work, please let me know. I am more than happy to fix it.