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

Parallel version #176

Closed
mariavd opened this issue Apr 25, 2018 · 7 comments
Closed

Parallel version #176

mariavd opened this issue Apr 25, 2018 · 7 comments

Comments

@mariavd
Copy link
Contributor

mariavd commented Apr 25, 2018

This is not an issue but a question. I compiled with the --omp flag and submitted a calculation of the current density in 3D on 8 cores, specifying

export OMP_NUM_THREADS=8
export OMP_STACKSIZE=16M

Is it supposed to print that it indeed is running on 8 cores? The job is rather big, and ETA does not fit into the format specifier, so I won't be able to tell if it indeed ran faster than serial.

@heikef
Copy link
Contributor

heikef commented Apr 26, 2018

You can also add --blas to speed it up when compiling.

@heikef
Copy link
Contributor

heikef commented Apr 26, 2018

#170

@bast
Copy link
Member

bast commented Apr 26, 2018

Right, the timing is not correct in case of OMP. As to BLAS, it is very important to use optimized blas, most probably MKL but for this we need to fix GIMIC issues on Intel.

@mariavd
Copy link
Contributor Author

mariavd commented Apr 26, 2018

Ok, thanks. Possibly it might be useful to print an extra line with the number of threads in use but this is of course just a suggestion.

@mariavd
Copy link
Contributor Author

mariavd commented May 16, 2018

Just to let you know that I added a line to print the ETA in hours in my fork. It looks like this:

 INFO: Estimated CPU time for single core calculation:  4781999.10 sec (1328.3 h )

I could make a pull request in case you find it useful. You don't have to, of course, since it's barely significant.

@bast
Copy link
Member

bast commented Jul 12, 2018

I think this is useful.

@mariavd
Copy link
Contributor Author

mariavd commented Aug 20, 2018

This issue should have been closed by #178. My fault that I didn't mark it.

@mariavd mariavd closed this as completed Aug 20, 2018
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

No branches or pull requests

3 participants