-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restructure ReservoirSimulationTimeSeries
#698
Comments
Quick summary from meeting between @asnyv, @jorgenherje and myself going through the linked issues above:
Related factory responsibility (@sigurdp):
|
Above mentioned summary as checklist (for @jorgenherje):
Related factory responsibility (@sigurdp):
|
Refactoring done. Two bullet points has gotten new issues: |
Co-authored-by: Shadab Khan <[email protected]>
The current plugin has "been with us" since the start, and has been extended by different people during the project period. Time to consolidate/restructure/improve the code.
Related issues:
ReservoirSimulationTimeSeries
can take in list of vectors instead ofvector{1,2,3}
as user input #677Functionality for calculating new custom vectors, delta between ensembles etc. should be in separate modules (as other plugins will need the same calculation options).
The text was updated successfully, but these errors were encountered: