-
Notifications
You must be signed in to change notification settings - Fork 666
Fb_global_unstr #335
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
Fb_global_unstr #335
Changes from 23 commits
98c5702
71b9ba9
6c42d17
5c361aa
cc34122
33a7af3
68c3c96
663d983
d4887f5
6295b14
dc0cea9
083a910
8917422
febddae
6bc0e5f
4ee6dc9
558e7bf
7a7d90a
77d513b
0ca3076
ef2da97
2ff1b57
a487bdb
69ca74a
e443b62
4177081
3a51cdf
af86335
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -135,8 +135,9 @@ | |
|
|
||
| # GRID STRT BOUND OUTF OUTP TRCK GRIB GINT GX* UPRSTR LIBWW3 | ||
| # reg_programs = non-NetCDF programs | ||
| reg_programs="ww3_grid" | ||
| reg_programs="$reg_programs ww3_strt" | ||
| # reg_programs="ww3_grid" | ||
| # reg_programs="$reg_programs ww3_strt" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. clean up commented out text
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done |
||
| reg_programs="ww3_strt" | ||
| reg_programs="$reg_programs ww3_bound" | ||
| reg_programs="$reg_programs ww3_outf" | ||
| reg_programs="$reg_programs ww3_outp" | ||
|
|
@@ -154,6 +155,7 @@ | |
| cdf_programs="$cdf_programs ww3_ounp" | ||
| cdf_programs="$cdf_programs ww3_bounc" | ||
| cdf_programs="$cdf_programs ww3_trnc" | ||
| cdf_programs="$cdf_programs ww3_grid" | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there an option that says if ww3_grid should be a netcdf program or not? The way this currently is, you are now requiring netcdf for ww3_grid, which essentially is now requiring netcdf as a library, no? While I know there are ongoing discussions of integrating more netcdf, is there a way to make this optional or is it always essential now?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @JessicaMeixner-NOAA It is required when SCRIPNC is in the switch. So, as Steven suggested, I can exclude SCRIP related developments, and we no longer need netcdf for ww3_grid, but I thought it might be helpful.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are all of the netcdf SCRIP related updates behind a switch?
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What I don't want to do, is require everyone even someone who is using all ascii files to now have a netcdf dependency. If we move all output to netcdf, then yes all programs will have netcdf dependency and it will be a requirement. Even in the make_makefile script, this looks like an option, not a requirement, so when deciding if ww3_grid is a netcdf or regular program, shouldn't this also be an option?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I realized it when I was checking mww3_test_04 with switch_PR3_UQ_MPI_SCRIPNC, so I had to modify the make_makefile.sh, the rest of tests went well without netcdf dependency, so I think these are behind SCRIPNC if I am not wrong.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this needs to be moved into the if/else section starting on line 172: https://github.com/aliabdolali/WW3/blob/FB_Global_Unstr/model/bin/w3_make#L172 If it's scripnc then add ww3_grid to netcdf programs, if not, it should be a regular program.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @JessicaMeixner-NOAA Thanks, it was a very helpful hint, I made the changes and ran the regtests again, all in good shape. Could you approve the review? |
||
|
|
||
| # PRTIDE | ||
| if [ -n "`grep TIDE $switch_file`" ] | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.