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

Additional NUOPC_Connector metadata #300

Open
theurich opened this issue Sep 19, 2024 · 5 comments
Open

Additional NUOPC_Connector metadata #300

theurich opened this issue Sep 19, 2024 · 5 comments
Assignees
Labels
feature/enhancement New feature or request NUOPC

Comments

@theurich
Copy link
Member

There are a few more settings we need to support on the generic NUOPC_Connector via attributes:

  • A flag to disable VMEpoch usage. It cannot just be ON/OFF, because NUOPC_Connector makes its own decision on whether VMEpoch can be turned ON/OFF (according to the limitation that VMEpoch is currently supported for disjoint send/recv petLists). But this option should allow the user to disable any usage of VMEpoch, even if NUOPC_Connector could turn it ON. On the other hand the option should not suggest to the user that they have a way to force the usage of VMEpoch.
  • We already have the EpochThrottle option exposed. We need to also expose the EpochEnterKeepAlloc and EpochExitKeepAlloc

In the ConnectionOptions attribute

  • We already have dumpWeights. We need something like dumpCommMatrix to allow dumping of the communication matrix that is available on each RH. Not sure exactly how filename is specified here... need to think about that also for dumpWeights.
@theurich theurich added feature/enhancement New feature or request NUOPC labels Sep 19, 2024
@theurich theurich self-assigned this Sep 19, 2024
@anntsay
Copy link

anntsay commented Sep 25, 2024

@danrosen25 FYI

@danrosen25
Copy link
Member

@anntsay
This looks like connector metadata being used internally. I'm not sure this is field metadata. Is it?

@anntsay
Copy link

anntsay commented Sep 25, 2024

Hi Dan, Got it. thanks for pointing that out. so basically this sounds like it is to expand the NUOPC_Connector attributes. so i think you are right. @theurich could you confirm?

@theurich
Copy link
Member Author

Yes, I just want to add a few more items to the existing Connection Options approach to provide NUOPC access to existing ESMF functionality.

@anntsay
Copy link

anntsay commented Oct 2, 2024

Gerhard: is motivated to have this in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/enhancement New feature or request NUOPC
Projects
None yet
Development

No branches or pull requests

3 participants