Relative wind #213
Open
Relative wind #213
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…efile accordingly. Added the parameter in the hycom_opt namelist file and updated the exp_preprocess to copy it in SCRATCH.
|
This has been included in the upstream version of HYCOM as of august this year. I suggest that I bring that into version 2.3. And leave out this version. It is implemented the same way. We can still implement this in HYCOM 2.2 |
|
Hi Till,
It would be a nice contribution to V2.3, and welcome from my point.
Is there any difference between the upstream update and our tested implementation?
Cheers,
Jiping
… On 18 Sep 2024, at 12:12, TRasmussen ***@***.***> wrote:
This has been included in the upstream version of HYCOM as of august this year. I suggest that I bring that into version 2.3. And leave out this version. It is implemented the same way.
@AntoineBernigaud <https://github.com/AntoineBernigaud> @Jipingnersc <https://github.com/Jipingnersc>
Any thoughts on this?
We can still implement this in HYCOM 2.2
—
Reply to this email directly, view it on GitHub <#213 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGDZNKIXUZS2YWEX7VLUSUTZXFG2NAVCNFSM6AAAAABONKUZNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGA3DINBYGQ>.
You are receiving this because you were mentioned.
|
|
Hello, I do not have much thoughts about it, it might be easier indeed to just take the upstream version of HYCOM into version 2.3 :) |
|
The only difference (as I see it) is that it is integrated into blkdat.input. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Clearer pull request for the relative wind changes in the code and the namelist file hycom_opt.