Skip to content

[Backport version-25.0] Skip RFT approximation for points outside the grid - #14358

Open
scout-team-app[bot] wants to merge 1 commit into
version-25.0from
backport-14277-to-version-25.0
Open

[Backport version-25.0] Skip RFT approximation for points outside the grid#14358
scout-team-app[bot] wants to merge 1 commit into
version-25.0from
backport-14277-to-version-25.0

Conversation

@scout-team-app

@scout-team-app scout-team-app Bot commented Sep 3, 2026

Copy link
Copy Markdown

Description

Backport of #14277 to version-25.0.

Observations whose location falls outside the grid have
well_connection_cell and well_connection_cell_center None. Attempting to
calculate an approximated response for a point with coordinates None
would raise a TypeError. Since responses and observations are matched on
well_connection_cell, an approximated response would not be matched to
any observation. Approximation is therefore skipped for such points.

(cherry picked from commit 03e5ba6)
@scout-team-app scout-team-app Bot added the release-notes:bug-fix Automatically categorise as bug fix in release notes label Sep 3, 2026
@codecov-commenter

codecov-commenter commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.90%. Comparing base (942c485) to head (2883b9e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@              Coverage Diff              @@
##           version-25.0   #14358   +/-   ##
=============================================
  Coverage         91.90%   91.90%           
=============================================
  Files               482      482           
  Lines             33467    33468    +1     
=============================================
+ Hits              30758    30760    +2     
+ Misses             2709     2708    -1     
Flag Coverage Δ
cli-tests 36.52% <0.00%> (-0.01%) ⬇️
fuzz 44.51% <0.00%> (-0.01%) ⬇️
gui-tests 58.79% <0.00%> (-0.01%) ⬇️
performance-and-unit-tests 80.77% <100.00%> (+<0.01%) ⬆️
test 46.04% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/ert/config/rft_config.py 96.44% <100.00%> (+0.36%) ⬆️

@codspeed-hq

codspeed-hq Bot commented Sep 3, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 36 untouched benchmarks


Comparing backport-14277-to-version-25.0 (2883b9e) with version-25.0 (942c485)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:bug-fix Automatically categorise as bug fix in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants