Skip to content

Add individual ERTBOX per zone#109

Merged
rnyb merged 2 commits into
mainfrom
testing
May 20, 2026
Merged

Add individual ERTBOX per zone#109
rnyb merged 2 commits into
mainfrom
testing

Conversation

@rnyb

@rnyb rnyb commented May 11, 2026

Copy link
Copy Markdown
Collaborator

Related to #103

This resolves the ERT part of the issue
The issue can be closed with next RMS update

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Drogon AHM ERT configuration to use zone-specific ERTBOX grids for APS field parameters, enabling different grid sizes per zone (as described in Issue #103) to reduce memory/disk/runtime when layer counts vary significantly between zones.

Changes:

  • Updated APS field parameter configuration to reference separate ERTBOX EGRID files per zone (Valysar/Therys/Volon) and bind each FIELD parameter to the appropriate grid.
  • Updated the main AHM ERT model include comment to reflect the new per-zone ERTBOX behavior.
  • Cleaned up the RMS project readme instructions for compress/restore and dev update steps.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
rms/model/0readme.txt Reformats/clarifies instructions for compressing/restoring the RMS project and documenting dev updates.
ert/model/drogon_ahm.ert Updates the comment on the APS include to indicate per-zone ERTBOX usage.
ert/input/config/ahm_field_aps.ert Introduces per-zone EGRID definitions and assigns each APS FIELD parameter to the correct zone-specific ERTBOX grid via GRID:.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rnyb
rnyb requested review from alifbe and oddvarlia May 11, 2026 12:08

@oddvarlia oddvarlia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks OK

@rnyb
rnyb merged commit 8e020a8 into main May 20, 2026
6 checks passed
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