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

Update building mapping files with moab #247

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Nov 27, 2024

This merge reorganizes support for generating mapping files using MOAB's mbtempest and related tools. This work relates to changes made by @bmooremaley in MPAS-Dev/compass#863.

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
  • Documentation has been built locally and changes look as expected
  • Testing comment in the PR documents testing used to verify the changes
  • New tests have been added to a test suite

@xylar xylar added enhancement New feature or request framework Changes relating to the polaris framework as opposed to individual tests or analysis labels Nov 27, 2024
@xylar xylar requested a review from bmooremaley November 27, 2024 11:47
@xylar xylar self-assigned this Nov 27, 2024
@xylar
Copy link
Collaborator Author

xylar commented Nov 27, 2024

I'm able to create a mapping file (very quickly) with this approach but I'm having trouble remapping 2D data with it because the 2 dimensions have been flattened into 1 in the mapping file. The solution may be to flatten the 2D data before remapping.

@xylar
Copy link
Collaborator Author

xylar commented Dec 4, 2024

The fix for the mapping problem above is:
https://bitbucket.org/fathomteam/moab/branch/vijaysm/propagate-rll-dims

@xylar
Copy link
Collaborator Author

xylar commented Dec 4, 2024

Thanks to @vijaysm for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request framework Changes relating to the polaris framework as opposed to individual tests or analysis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant