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

Nextsimdg test cycling DA on betzy #9

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Nextsimdg test cycling DA on betzy #9

wants to merge 18 commits into from

Conversation

myying
Copy link
Collaborator

@myying myying commented Feb 2, 2025

Based on previous PR #7, and PR #8 , I've tested nextsimdg model ensemble forecasts on betzy.sigma2.no computers. Found a few further issues and resolved in the following commits.

I'm now testing nextsimdg model DA code assimilating some OSISAF routine observations. Cherry-picking some development of the data assimilation code from "other_features" branch, will also include these commits in this PR.

hcorr is with unit (unlike in fortran code) so scorr here don't need to divide by dx
the namelist option "init_file" meant for the very first initial
condition file, not the restart file during cycling; to link the first
cycle restart file, we need to keep time_start in kwargs

during cycling, restart files will be copied over, not linked over,
since DA step will update the restart file, making it a new copy
restart at first cycle perturbed seems to cause forecast to fail,
removed perturb:restart option; only perturb the forcing for now.

add parallel_mode options in slurm.py job submitter, to allow running
openmp program in single-job mode (spawning multiple model.run in a big
allocation using srun). The job script is
scripts/nextsim.dg.test.submit.sh
only show the messages when c.debug=True
@myying myying requested review from yumengch and aydogduali February 2, 2025 10:29
myying and others added 10 commits February 2, 2025 21:04
if variable is in a group: e.g. data/tair, the assert on variable name in f.variables is not working. use try-catch instead f['data/tair'] shall work
renamed scripts/diag.py to scripts/diagnose.py
f.variables doesn't work for compound varname,
change the way to assert variable is in file.
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.

1 participant