-
Notifications
You must be signed in to change notification settings - Fork 131
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
add Pstar
and Cstar
to namelist
#475
add Pstar
and Cstar
to namelist
#475
Conversation
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.
Won't Icepack need to be changed, too? It seems like Icepack needs the strength parameters for ridging.
e
@eclare108213 |
Great, thanks. |
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.
I think this just needs units for the Pstar entry in ug_case_settings, otherwise it looks good.
Also, add a new format to ice_init for numbers in exponential notation. See CICE-Consortium#472
2120c82
to
b2633ec
Compare
thanks @eclare108213, I just added |
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.
This looks fine to me. I think I'll do some regression testing before merging though to make sure it's bit-for-bit. I expect it to be, but if not, it's good to know before the merge.
Regression testing looks fine, https://github.com/CICE-Consortium/Test-Results/wiki/cice_by_hash_forks#b2633ec3ef470a332d4e63d9d4b3736c9eeee12c. I will merge |
'Pstar' was added to the namelist in c9cfebd (add Pstar and Cstar to namelist (CICE-Consortium#475), 2020-06-26), but the wrong units were added to the documentation (N/m instead of N/m^2). Fix that.
'Pstar' was added to the namelist in c9cfebd (add Pstar and Cstar to namelist (CICE-Consortium#475), 2020-06-26), but the wrong units were added to the documentation (N/m instead of N/m^2). Fix that. Reported-by: Jean-François Lemieux <jean-francois.lemieux@canada.ca>
'Pstar' was added to the namelist in c9cfebd (add Pstar and Cstar to namelist (CICE-Consortium#475), 2020-06-26), but the wrong units were added to the documentation (N/m instead of N/m^2). Fix that. While at it, fix the RST syntax for `k2`. Reported-by: Jean-François Lemieux <jean-francois.lemieux@canada.ca>
'Pstar' was added to the namelist in c9cfebd (add Pstar and Cstar to namelist (CICE-Consortium#475), 2020-06-26), but the wrong units were added to the documentation (N/m instead of N/m^2). Also, it already appeared in the index with the same wrong units. Fix that. While at it, fix the RST syntax for `k2`. Reported-by: Jean-François Lemieux <jean-francois.lemieux@canada.ca>
* doc: correct 'Pstar' units 'Pstar' was added to the namelist in c9cfebd (add Pstar and Cstar to namelist (#475), 2020-06-26), but the wrong units were added to the documentation (N/m instead of N/m^2). Also, it already appeared in the index with the same wrong units. Fix that. While at it, fix the RST syntax for `k2`. Reported-by: Jean-François Lemieux <jean-francois.lemieux@canada.ca> * icepack: update to b1e41d9
PR checklist
Add
Pstar
andCstar
to namelistP. Blain
Tested that the new namelist values are correctly printed in the run log.