Skip to content

DON'T MERGE YET: Seaice DA, needs the GDASApp PR merged first. #12

Closed
guillaumevernieres wants to merge 7 commits into
JessicaMeixner-NOAA:feature/c384wcdafrom
guillaumevernieres:feature/c384wcda-seaice-da-gv
Closed

DON'T MERGE YET: Seaice DA, needs the GDASApp PR merged first. #12
guillaumevernieres wants to merge 7 commits into
JessicaMeixner-NOAA:feature/c384wcdafrom
guillaumevernieres:feature/c384wcda-seaice-da-gv

Conversation

@guillaumevernieres
Copy link
Copy Markdown

@guillaumevernieres guillaumevernieres commented May 6, 2024

  • We need to add a seaice analysis restarts to the IC's

Comment thread ush/forecast_postdet.sh Outdated
local seconds
seconds=$(to_seconds "${restart_date:8:2}0000") # convert HHMMSS to seconds
cice_restart_file="${DATArestart}/CICE_RESTART/cice_model.res.${restart_date:0:4}-${restart_date:4:2}-${restart_date:6:2}-${seconds}.nc"
if [[ "${DO_JEDIOCNVAR:-NO}" = "YES" && "${restart_date}" = "${sdate_current_cycle}" ]]; then
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guillaumevernieres Two things, one i changed the sdate_current_cycle name on you to be: model_start_date_current_cycle -- sorry!

And 2, I don't think you can have RERUN=YES if restart_date=model_start_date_current_cycle, so I don't think this is a needed block.... but we should get someone else to confirm.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arf ... yes, you also told me about it. I'll remove the block too.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you push that change @JessicaMeixner-NOAA ? I don't see model_start_date_current_cycle defined anywhere.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just pushed.. sorry i thought i had

@JessicaMeixner-NOAA
Copy link
Copy Markdown
Owner

Closing... This branch is now in develop.

JessicaMeixner-NOAA pushed a commit that referenced this pull request Apr 14, 2025
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.

2 participants