-
Notifications
You must be signed in to change notification settings - Fork 132
Ert has multiple endpoints for misfit data #12579
Copy link
Copy link
Open
Labels
improvementSomething nice to have, that will make life easier for developers or users or both.Something nice to have, that will make life easier for developers or users or both.maintenanceNot a bug now but could be one day, repaying technical debtNot a bug now but could be one day, repaying technical debt
Metadata
Metadata
Assignees
Labels
improvementSomething nice to have, that will make life easier for developers or users or both.Something nice to have, that will make life easier for developers or users or both.maintenanceNot a bug now but could be one day, repaying technical debtNot a bug now but could be one day, repaying technical debt
Type
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
No status
Currently, you can find calculations for misfits (at least) in the following places:
ert/src/ert/libres_facade.py
Lines 29 to 74 in 6681256
ert/src/ert/gui/tools/plot/plottery/plots/misfits.py
Lines 119 to 141 in 6681256
ert/src/ert/dark_storage/compute/misfits.py
Lines 8 to 42 in 6681256
We should create a single endpoint for calculating misfits to avoid multiple implementations and potential differences in misfit values.
It is sensible for the Ensemble to calculate it's own misfit, so creating an endpoint here and removing the others and change their usages to use the new one in Ensemble would be a huge improvement.