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

Provide lower resolution wave model grid & IC set-ups #1333

Closed
JessicaMeixner-NOAA opened this issue Feb 24, 2023 · 3 comments · Fixed by #1653
Closed

Provide lower resolution wave model grid & IC set-ups #1333

JessicaMeixner-NOAA opened this issue Feb 24, 2023 · 3 comments · Fixed by #1653
Assignees
Labels
duplicate This issue or pull request already exists

Comments

@JessicaMeixner-NOAA
Copy link
Contributor

Description
For efficient testing and development, a low resolution wave model set-up is needed within the workflow. In addition to grids, ICs are needed to be staged (Please let me know if there's already a specific date being used for other testing).

Requirements
Able to run waves within low resolution testing.

Acceptance Criteria (Definition of Done)
Merged in the develop branch and can run low-res fully coupled tests for whatever configurations are currently being tested.

Dependencies
The biggest blocking issue here is my time.

@JessicaMeixner-NOAA JessicaMeixner-NOAA self-assigned this Feb 24, 2023
@JessicaMeixner-NOAA
Copy link
Contributor Author

FYI @aerorahul -- If there is a specific date for ICs please let me know.

@aerorahul
Copy link
Contributor

Is #259 related to this issue?

@JessicaMeixner-NOAA
Copy link
Contributor Author

Good catch @aerorahul -- I would consider these more/less the same.

@aerorahul aerorahul added the duplicate This issue or pull request already exists label Feb 27, 2023
WalterKolczynski-NOAA pushed a commit that referenced this issue Jun 1, 2023
This adds an option for 2 or 5 deg global wave models, which will provide a light weight version to assist in engineering development.    
-- This PR requires PR #1648 to be merged as this builds on it as well as issue #1652 which adds the fix files needed for this PR to be addressed first. 

The resources were chosen to stay under  1 or 2 node and be faster than the atm component.  

- Fixes #1333
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants