-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add OpenMP directives #453
Comments
@EricEngle-NOAA is this still a work in progress? What went wrong with your PR? |
Any progress on this? SHould I bump to next release? |
No progress. Please table for the next release. |
We are now working more closely with the wgrib2 project, which (I think) has openmp everywhere. So it might be worth comparing codes. We can ask @webisu for lessons learned, or, even better, just cut and paste his already-working code! ;-) I have bumped this to the next release. |
There are several places where OpenMP directives can be added for faster performance -- mainly in the packing and unpacking functions were data are scaled (unscaled) and packed (unpacked).
Introducing this would also mean changes to the CMake configuration files so the OpenMP library can be linked.
The text was updated successfully, but these errors were encountered: