-
Notifications
You must be signed in to change notification settings - Fork 568
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
Bugfix/st4wcf #261
Bugfix/st4wcf #261
Conversation
fixed file name for ww3_ounf output See merge request wave/WW3!1
bugfix for netcdf file id allocation See merge request wave/WW3!2
extra line in ww3_shel See merge request wave/WW3!4
add comment
Feature/fixedname See merge request wave/WW3!5
add WCOR to switch list See merge request wave/WW3!6
add check on array index See merge request wave/WW3!7
Merge branch 'develop' of https://gitlab.ifremer.fr/wave/WW3 into develop
…into feature/ST4tail Conflicts: model/ftn/w3gdatmd.ftn model/ftn/w3src4md.ftn model/ftn/ww3_grid.ftn
…into feature/ST4tail
to integrate minor changes lost by past mistakes with git commands
it was due to missing variables in out_grd and missing initialization of some variables
Great work Mickael - well done on finding that bug. |
I've added the matrix results |
ww3_ta1 and ww3_ts1 are still showing differences in the out_grd.ww3 files (which is expected if you are comparing against the original develop branch, of course). Just to confirm - are these tests now self consistent with each other? |
I've compared the figures against the ones from the develop branch in April 21st (before st4tail merge) and I get the similar plots some differences due to the new parameterization. Does this reply to your question ? (it's also now b4b reproducible on my side) |
Yes - thank you Mickael. |
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.
All good to go!
Thanks @mickaelaccensi |
description :
bugfix for ST4 b4b reprodctibility
linked to issue #249
model :
w3iogrmd : write/read variables AIRCMIN and AIRGB in mod_def.ww3
w3src4md : initialize array
update VERXXX in w3iogrmd
matrix :
done with intel-comp/18, impi/2018.1.163, netCDF-Fortran 4.5.2
ww3_ts1 : results are correct now
ww3_ta1 : results are correct now
ww3_tp2.14 : differences due to metadata history in remap file (not relevant)
ww3_tp2.18 : now differences due to output messages order (dedicated bugfix in progress)
mww3_test_03 : expected not b4b results
mww3_test_04 : different dates in metadata 'history' (it will be skipped in another bugfix branch)
matrix_ST4wcf.zip