equinor / equinor/ert

Add new storage migration for ertbbox params in Fields

Open
#12,263 1 comment 0 reactions 0 assignees View on GitHub
blocked maintenance payback
Dominant language
Python
Stars
161
Forks
140
Avg merge
2d 4h
Merged PRs (30d)
138

Description

Currently there are ert box params:
```python
xlength: float | None = None
ylength: float | None = None
xinc: float | None = None
yinc: float | None = None
rotation_angle: float | None = None
origin: tuple[float, float] | None = None
```

which when migrated from the older version of storage, these will get assigned `None`. This might end up in a wrong storage state. Therefore we should do an extra storage migration which we populate those values correctly if possible ofc.

Contributor guide

Open the contributing guide

Research direction

Locate the Fields storage-migration entry point and compare the older storage representation with the six ert box parameters listed in the issue. Determine which values can be recovered, then verify that migrating older storage populates recoverable values instead of leaving them as None.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
database
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.