bp / bp/resqpy

Transmissibility uom

Open
#144 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

Flow simulators make transmissibility independent of the fluid viscosity by including it as a term in the uom, eg: m3.cP/(kPa.d). The nearest uom in the Energistics list does not contain a viscosity term, eg. m3/(kPa.d).

Resqpy currently generates a uom of the first form when calculating transmissibility properties. The new, stricter, weights_and_measures module will presumably reject such a uom.

The ideal resolution of this would be for Energistics to include the viscosity term in the uoms intended for transmissibility, to align with flow simulators. (The Energistics data does include a definition of transmissibility which seems to align with the flow simulators: "Volumetric flux per unit area per unit pressure drop for unit viscosity fluid")

Contributor guide

Open the contributing guide

Research direction

Start by reading the transmissibility property calculation and the weights_and_measures module, then compare the generated m3.cP/(kPa.d) unit with the nearest Energistics unit listed in the issue. Check how the stricter module handles the generated unit. Done requires a decided, compatible unit representation, but the issue notes that the ideal resolution depends on Energistics.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.