-
Notifications
You must be signed in to change notification settings - Fork 32
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
Test with ESMF 8.1 #82
Comments
Just from some trivial testing with our own workloads, it seems considerably slower than 8.0.1. Could we have MOAB exposed through xESMF to try? |
Is My guess is that you'd first need to include it here: https://github.com/esmf-org/esmf/blob/develop/src/addon/ESMPy/src/ESMF/interface/cbindings.py |
@huard @Plantain we have created a feature request for this under #37. |
Is that a private repo? I can't see it.
…On Thu, Apr 22, 2021 at 12:09 PM rokuingh ***@***.***> wrote:
@huard <https://github.com/huard> @Plantain <https://github.com/Plantain>
we have created a feature request for this under #37
<https://github.com/esmf-org/esmf-support/issues/37>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#82 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACODF32OZZSJAZ4SPVD4P3TJ6AM5ANCNFSM42B6MWCQ>
.
|
Oops, my mistake, it is. We are just moving our issues over to github and still have it set to private. Well, for now just know that we are aware of the issue. If you could let us know what the urgency is on this request it will help us to prioritize it for the upcoming release cycle. |
Well, I don't know about urgent but I think it should be a hard block on
making MOAB the default if all the users of ESMF via ESMPy have not been
able to test their workflows in advance?
…On Thu, Apr 22, 2021 at 12:19 PM rokuingh ***@***.***> wrote:
Oops, my mistake, it is. We are just moving our issues over to github and
still have it set to private. Well, for now just know that we are aware of
the issue. If you could let us know what the urgency is on this request it
will help us to prioritize it for the upcoming release cycle.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#82 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACODF5CCGF3SIXGY3RB5BDTJ6BU7ANCNFSM42B6MWCQ>
.
|
agreed. we don't plan to have MOAB be the default backend to ESMF_Mesh until 8.3.0, two releases from now, about a year out at best guess. This ticket is just to add a flag to allow a user to toggle the MOAB mesh functionality. |
A feature request can be submitted with an email to esmf_support@ucar.edu. |
xESMF now tested with a range of ESMF versions, see #230 |
A new release of ESMF is available. We should
https://github.com/esmf-org/esmf/releases/tag/ESMF_8_1_0
The text was updated successfully, but these errors were encountered: