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

Ww3 hycom #926

Draft
wants to merge 169 commits into
base: develop
Choose a base branch
from
Draft

Ww3 hycom #926

wants to merge 169 commits into from

Conversation

heloisemichaud
Copy link

@heloisemichaud heloisemichaud commented Mar 2, 2023

Pull Request Summary

adaptation to the hycom coupling

Description

Issue(s) addressed

Commit Message

Check list

Testing

  • How were these changes tested?
  • Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
  • Have the matrix regression tests been run (if yes, please note HPC and compiler)?
  • Please indicate the expected changes in the regression test output, (Note the list of known non-identical tests.)
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):

@JessicaMeixner-NOAA
Copy link
Collaborator

Hi @heloisemichaud Thanks for this PR. This PR seems to be either build on or be a repeat of PR #888. Can you clarify? Also, we will need you to fill out the entire PR template.

@aronroland
Copy link
Collaborator

aronroland commented Mar 2, 2023 via email

@JessicaMeixner-NOAA
Copy link
Collaborator

@aronroland Happy to help! @heloisemichaud - just reply back with any questions you have - we're happy you've made a PR!

@heloisemichaud heloisemichaud marked this pull request as draft March 2, 2023 13:27
@aronroland
Copy link
Collaborator

just some clarification this will be a addition to the OASIS coupler, which is only dedicated when ww3 is coupled to hycom, whereas #888 is more general. Heloise and me will do some more training tomorrow and hopefully finish the pull request. Thanks for your understanding.

@MatthewMasarik-NOAA
Copy link
Collaborator

Hello @aronroland, I wanted to check in to see if there is anything needed from us for this PR? There is no rush of course if you continue to work on it, I just wanted to touch base on it.

@aronroland
Copy link
Collaborator

@MatthewMasarik-NOAA i will discuss with Heloise and provide some time line, this should be closed pretty soon. Thanks for pointing this out.

@MatthewMasarik-NOAA
Copy link
Collaborator

Thanks for the update, @aronroland. Please reach out if we can be of any help

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.

6 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy