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

Update UGWD files for new grid files and new orography statistics method #2316

Closed
2 of 4 tasks
barlage opened this issue Feb 14, 2024 · 12 comments
Closed
2 of 4 tasks
Assignees

Comments

@barlage
Copy link

barlage commented Feb 14, 2024

Target fix directory

ugwd

Type of change

  • Update existing files
  • Remove files
  • Add new files to existing directory
  • Add new fix directory

Any workflow changes needed?

Fix version only

Related issues

The ugwd files are updated in two ways:

Using the same directory timestamp (pickup location below) as #1981

Pickup location

/scratch2/NCEPDEV/land/data/fix/ugwd/20231027

Size

0

@KateFriedman-NOAA
Copy link
Member

@barlage I have copied the provided files to a new 20231027 timestamp subfolder of fix/ugwd. Please see here on Hera and confirm this is in line with your request: /scratch1/NCEPDEV/global/glopara/fix/ugwd/20231027

@JessicaMeixner-NOAA
Copy link
Contributor

I've added this to my PR: #2230

I did a quick diff between @barlage directory and the glopara, and while i'm not the expert, I at least can independently confirm there are no differences between /scratch1/NCEPDEV/global/glopara/fix/ugwd/20231027 and /scratch2/NCEPDEV/land/data/fix/ugwd/20231027

@JessicaMeixner-NOAA
Copy link
Contributor

@barlage I have copied the provided files to a new 20231027 timestamp subfolder of fix/ugwd. Please see here on Hera and confirm this is in line with your request: /scratch1/NCEPDEV/global/glopara/fix/ugwd/20231027

@KateFriedman-NOAA I see that @barlage is in meetings all day. I'm unsure how much effort it is to sync across platforms. However, we need to get new fix files to run the last set of high resolution tests for #2230 to then start the last set of CI tests for that. If it's not too much work can we just sync these to other machines? If it's a lot of work, I am 100% in favor of waiting for the green light from Mike, just thought I'd ask.

@WalterKolczynski-NOAA
Copy link
Contributor

WalterKolczynski-NOAA commented Feb 15, 2024

I'm synchronizing to Orion now so we can get tests rolling on the assumption it is correct.

@JessicaMeixner-NOAA
Copy link
Contributor

I'm synchronizing to Orion now so we can get tests rolling on the assumption it is correct.

Thanks @WalterKolczynski-NOAA !

@KateFriedman-NOAA
Copy link
Member

I'm syncing to the WCOSS2s right now (Dogwood first).

@JessicaMeixner-NOAA
Copy link
Contributor

Thank you @KateFriedman-NOAA !

@KateFriedman-NOAA
Copy link
Member

The new 20231027 UGWD fix files are now available on both WCOSS2s.

@barlage
Copy link
Author

barlage commented Feb 15, 2024

Thanks for starting without me, haven't had much time to check email today. I did some quick checks and everything looks good to me. Thanks @JessicaMeixner-NOAA for doing some independent checks.

@KateFriedman-NOAA
Copy link
Member

Is this issue now complete or will an upcoming PR close it? Thanks!

@JessicaMeixner-NOAA
Copy link
Contributor

This was updated with #2230 which was merged last night. I think this issue can be closed. (I forgot to add this issue to that PR so it did not automatically close).

@KateFriedman-NOAA
Copy link
Member

Great, thanks for confirming! Closing as complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants