bp / bp/resqpy

rq_import functions need pressure uom argument

Open
#858 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
67
Forks
18
Avg merge
2h 26m
Merged PRs (30d)
1

Description

The rq_import functions for importing Nexus data do not allow for a pressure unit of measure to be specified when importing properties from vdb. The uom is guessed by the existing code, unreliably. The import functions such as import_nexus() and import_vdb_all_grids() need a new argument to specify the pressure uom implicit in the vdb data.

Nexus is a trademark of Halliburton.

Contributor guide

Open the contributing guide

Research direction

Start at the import_nexus() and import_vdb_all_grids() entry points and trace how pressure units are currently inferred while importing Nexus data from vdb. Add a way for callers to specify the pressure unit of measure, and verify that the supplied unit controls the imported properties instead of unreliable guessing.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.