Skip to content

ENH: nested grid with optional upscaling mappings#439

Open
lahus-0 wants to merge 1 commit into
equinor:mainfrom
lahus-0:nested_upscale
Open

ENH: nested grid with optional upscaling mappings#439
lahus-0 wants to merge 1 commit into
equinor:mainfrom
lahus-0:nested_upscale

Conversation

@lahus-0

@lahus-0 lahus-0 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Add optional input of upscaling mappings from a geogrid to the input grid. These are then updated to map to the resulting nested hybrid grid. For situations where the geogrid and input grid are not perfectly aligned in structure due to the differences in resolution. This ensures that a proper mapping between cells in geogrid to nested hybrid grid can be maintain when both are originally sourced from same grid.

Main documentation will likely include the addition of an example workflow in drogon after this is all finalised.

Checklist

  • Tests added (if not, comment why)
  • Test coverage equal or up from main (run pytest with --cov=src/ --cov-report term-missing)
  • If not squash merging, every commit passes tests
  • Appropriate commit prefix and precise commit message used
  • All debug prints and unnecessary comments removed
  • Docstrings are correct and updated
  • Documentation is updated, if necessary
  • Latest main rebased/merged into branch
  • Added comments on this PR where appropriate to help reviewers
  • [?] Moved issue status on project board
  • Checked the boxes in this checklist ✅

@lahus-0
lahus-0 marked this pull request as draft June 5, 2026 10:09
@lahus-0
lahus-0 marked this pull request as ready for review June 5, 2026 17:28
@lahus-0

lahus-0 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

I'll merge all the other recent changes on nested hybrid grid setups with this and update next week to resolve the conflicts.

@lahus-0

lahus-0 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

merged with all the other changes to nested hybrid grid setup.

@lahus-0

lahus-0 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

adapted to fit with change of NestedHybridGrid into a class

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.

1 participant