Skip to content
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

Merged
merged 55 commits into from
Oct 23, 2020
Merged

Bugfix/st4wcf #261

merged 55 commits into from
Oct 23, 2020

Conversation

mickaelaccensi
Copy link
Collaborator

@mickaelaccensi mickaelaccensi commented Oct 20, 2020

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

mickaelaccensi and others added 30 commits May 21, 2019 07:00
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
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
@mickaelaccensi mickaelaccensi marked this pull request as draft October 20, 2020 08:16
@ukmo-ccbunney
Copy link
Collaborator

Great work Mickael - well done on finding that bug.
I'll approve once you have the full regression matrix results.

@mickaelaccensi mickaelaccensi marked this pull request as ready for review October 21, 2020 07:54
@mickaelaccensi
Copy link
Collaborator Author

I've added the matrix results

@ukmo-ccbunney
Copy link
Collaborator

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?

@mickaelaccensi
Copy link
Collaborator Author

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)

@ukmo-ccbunney
Copy link
Collaborator

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.

Copy link
Collaborator

@ukmo-ccbunney ukmo-ccbunney left a 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!

@aliabdolali
Copy link
Contributor

Thanks @mickaelaccensi
I checked the regtests on NOAA HPC with intel, impi and netcdf
matrix_bugfix_ST4wcf.zip

@aliabdolali aliabdolali merged commit 9735c7d into NOAA-EMC:develop Oct 23, 2020
@mickaelaccensi mickaelaccensi deleted the bugfix/ST4wcf branch November 2, 2020 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants