Skip to content

Add mesh block au region mapping#7829

Merged
micholm merged 5 commits intomainfrom
add-mesh-block-au-region-mapping
Apr 21, 2026
Merged

Add mesh block au region mapping#7829
micholm merged 5 commits intomainfrom
add-mesh-block-au-region-mapping

Conversation

@micholm
Copy link
Copy Markdown
Contributor

@micholm micholm commented Apr 20, 2026

add region mapping for mesh blocks
image

What this PR does

Fixes #

This is a description of what I've done in this PR, especially explaining choices made to make the reviewer's job easier. If I don't replace this paragraph with an actual description, my PR will probably be ignored.

Test me

How should reviewers test this? (Hint: If you want to provide a test catalog item, create a Gist of its catalog JSON, add its url and your branch name to this url: http://ci.terria.io/<branch name>/#clean&<raw url of your gist> , and then paste that in the body of this PR)

Checklist

  • There are unit tests to verify my changes are correct or unit tests aren't applicable (if so, write quick reason why unit tests don't exist)
  • I've updated relevant documentation in doc/.
  • I've updated CHANGES.md with what I changed.
  • I've provided instructions in the PR description on how to test this PR.

Copy link
Copy Markdown
Collaborator

@na9da na9da left a comment

Choose a reason for hiding this comment

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

LGTM.

Tests failing, maybe we need to increment this constant to 155?

Might be good to add a line to CHANGES.md.

@micholm
Copy link
Copy Markdown
Contributor Author

micholm commented Apr 21, 2026

LGTM.

Tests failing, maybe we need to increment this constant to 155?

Might be good to add a line to CHANGES.md.

Ohh totally forgot about this const. Thanks :)

@micholm micholm merged commit 2d6577e into main Apr 21, 2026
10 checks passed
@micholm micholm deleted the add-mesh-block-au-region-mapping branch April 21, 2026 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants