Skip to content

Support for cube mesh api [AVD-1663]#4064

Merged
bjlittle merged 7 commits intoSciTools:mesh-data-modelfrom
pp-mo:cube_meshcoords
Mar 17, 2021
Merged

Support for cube mesh api [AVD-1663]#4064
bjlittle merged 7 commits intoSciTools:mesh-data-modelfrom
pp-mo:cube_meshcoords

Conversation

@pp-mo
Copy link
Member

@pp-mo pp-mo commented Mar 12, 2021

🚀 Pull Request

Description

Adding additonal mesh-specific Cube support:

  • add cube.mesh() and cube.location() access methods
  • make cube.__eq__ compare mesh + location
  • adjust all cube creation + coord management to ensure any/all MeshCoords are always inter-compatible

Reference #4063.


Consult Iris pull request check list

@pp-mo pp-mo marked this pull request as ready for review March 15, 2021 14:41
@pp-mo pp-mo force-pushed the cube_meshcoords branch from ae95431 to 15034b0 Compare March 15, 2021 14:45
Copy link
Member

@bjlittle bjlittle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pp-mo It's almost there... nice one! 👍

Just a few outstanding review comments to address. Hope you don't mind 😄

Copy link
Member Author

@pp-mo pp-mo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?? I seem to be blocked here by something + it thinks I am writing a review.
Submit/completing that to see if it will sort itself out...

@bjlittle bjlittle merged commit 04e71c1 into SciTools:mesh-data-model Mar 17, 2021
@pp-mo pp-mo deleted the cube_meshcoords branch March 18, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants