-
Notifications
You must be signed in to change notification settings - Fork 553
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
Pull request for compiling shared objects. (#47) #65
Conversation
@flampouris this pull request is next in line for updating the develop branch. Keep tuned this should happen in the next few days (+16h of full regtests matrix + weekend ahead). |
…mer develop merge.
Completed regtests with following results on NCEP R&D machine Theia with module load intel/14.0.2 impi/5.1.2.150 netcdf/4.3.0 Results confirm previous output from regtests of pull requests merged to develop where some expected differences were found in a very small number (11/582) of test cases (issue under investigation). Summary of cases with differences: ********************* non-identical cases **************************** ******************** summary of comparison ***************************
found 135 files in base directory
found 135 files in base directory
found 135 files in base directory
found 135 files in base directory
found 135 files in base directory
found 135 files in base directory
found 135 files in base directory
found 25 files in base directory
found 60 files in base directory
found 78 files in base directory
found 13 files in base directory |
@ukmo-ccbunney and @mickaelaccensi please review this pull request using the previous comment with output from the regtests matrix and examining the code changes, and indicate approval or request changes. |
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.
it's ok for me
Thanks @mickaelaccensi . @ukmo-ccbunney please let me know if you were able to review this pull request. Thanks! |
Hi @JessicaMeixner-NOAA please review the PR as per recent discussion. |
@JessicaMeixner-NOAA @aliabdolali @thesser1 @ukmo-ccbunney @ukmo-ansaulter @mickaelaccensi I have updated the develop branch after merging this pull request. Please update your repositories accordingly |
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.
@ajhenrique - Apologies for the late response regarding the review request (I've been on A/L).
I've had a look at the changes and tested the compilation of libww3.so; it looks ok to me.
Optional compilation of shared objects. Creates libww3.so. From @flampouris #47.