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/ounp #263

Merged
merged 54 commits into from
Oct 23, 2020
Merged

Bugfix/ounp #263

merged 54 commits into from
Oct 23, 2020

Conversation

mickaelaccensi
Copy link
Collaborator

@mickaelaccensi mickaelaccensi commented Oct 20, 2020

description :
bugfix for ounp string40
linked to issue #262

model :
ww3_ounp : correct missing statements for 40 characters names

matrix :
done with intel-comp/18, impi/2018.1.163, netCDF-Fortran 4.5.2
all : changes in output netcdf from string16 to string40
mww3_test_03 : not b4b known
matrix_ounp.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
Fabrice Ardhuin and others added 22 commits March 6, 2020 12:53
to integrate minor changes lost by past mistakes
with git commands
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.

Changes look good to me Mickael.

With referecnce to the CHECK_ERR changes, I added the following to ww3_ounf in a branch that I am working on:

#define CHECK_ERR(I) CHECK_ERROR(I, __LINE__)

and renamed the subroutine to CHECK_ERROR. It saves you from having to provide specific value for ICODE in your modified subroutine, but obviously gives the line number in the pre-processed code (which is fine for debugging during development). Just thought I would mention it in case you were interested.

@mickaelaccensi
Copy link
Collaborator Author

Changes look good to me Mickael.

With referecnce to the CHECK_ERR changes, I added the following to ww3_ounf in a branch that I am working on:

#define CHECK_ERR(I) CHECK_ERROR(I, __LINE__)

and renamed the subroutine to CHECK_ERROR. It saves you from having to provide specific value for ICODE in your modified subroutine, but obviously gives the line number in the pre-processed code (which is fine for debugging during development). Just thought I would mention it in case you were interested.

ok, I think we will need to upgrade the CHECK_ERR function to yours in all the WW3 codes

@aliabdolali
Copy link
Contributor

Thanks @mickaelaccensi
I also checked it on NOAA HPC with intel, impi, netCDF
matrix_bugfix_ounp.zip

@aliabdolali aliabdolali merged commit 03e13f7 into NOAA-EMC:develop Oct 23, 2020
@mickaelaccensi mickaelaccensi deleted the bugfix/ounp branch November 2, 2020 13:54
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