-
Notifications
You must be signed in to change notification settings - Fork 4
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
Merge OpenMP branch to master #8
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In a test run using a common input data file and same command line parameters, the output data files from the current master and the dev_omp_merged branch seem to not match with each other. The comparison was done using the 'diff' command. The variable indicating the number of threads was not used, so the default number of threads were used in this test run.
I'll have a look soon then. It must be that something got mixed up during merge. |
The issue has been fixed. The problem was related to the introduction of the variable |
67150b5
to
602f65e
Compare
This pull request includes addition of the option to utilise OpenMP (or not) to parallelize a number of compute intensive code sections.