Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wind_save very slow #102

Closed
Higginbottom opened this issue Aug 28, 2014 · 2 comments
Closed

wind_save very slow #102

Higginbottom opened this issue Aug 28, 2014 · 2 comments
Labels

Comments

@Higginbottom
Copy link
Collaborator

NSH has been carrying out profiling work on python in support of a HPC request for the work with DP. Specificially, I have been trying to find out how many CPUs are worth throwing at one python job. I discovered that more than 200 (for the PK04 geometry) was pointless, since the overall execution time does not reduce. Why? It is the act of saving the wind file after each ionization cycle. For example, with 100 processors, the entire photon generation, transport and ionization calculations takes about 1000 seconds. Then outputting the wind file takes about another 1000 seconds. Either this is a bug, and this wind save process needs to be seriously streamlined, or it is simply the way it is, and to my mind we need to make this step optional so we can take advantage of much bigger machines..

@Higginbottom
Copy link
Collaborator Author

OK, so I think this is a bug. I notice that the command to save the wind after each ionization cycle is not inside a MPI check statement. Therefore, 200 processes are all trying to save a wind file, to the same wind file, at the same time. This is not good IMHO. I put the wind save command inside a ifdef MPI_ON check, and only write out if I'm process 0. This fixes it.

@jhmatthews
Copy link
Collaborator

Good spot Nick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants