Skip to content
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

Closed
anders-kiaer opened this issue Jun 28, 2021 · 3 comments
Closed

Restructure ReservoirSimulationTimeSeries #698

anders-kiaer opened this issue Jun 28, 2021 · 3 comments
Assignees
Labels

Comments

@anders-kiaer
Copy link
Collaborator

anders-kiaer commented Jun 28, 2021

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:

Functionality for calculating new custom vectors, delta between ensembles etc. should be in separate modules (as other plugins will need the same calculation options).

@anders-kiaer
Copy link
Collaborator Author

Quick summary from meeting between @asnyv, @jorgenherje and myself going through the linked issues above:

  • Create a common utility to create fan chart input to plotly dcc.Graph.

    • Make mean independent of the shades (but keep in same legend group).
  • Try to fetch user selected date range in plotly graph and use it when changing to new vector.

  • Create UI (and later support it in the plugin) for showing multiple ensembles (with varying A and B).

    • @jorgenherje to make some UI sketches before implementing.
  • Support user defined vector annotations as input to plugin.

    • E.g. user defined descriptions of user defined UDQ.
    • E.g. user defined annotations ([{vector, datetime, label}]).
  • Stacking possibility (i.e. show time series in same plot)

    • across ensembles enforces same vector
    • if multiple vectors enforces same ensemble.
  • Statistics + individual realizations in same plot.

  • Secondary axis (and/or third axis?)

    • Automatically decide if same axis or different axis based on "same unit and not order of magnitude difference in max/min).

Related factory responsibility (@sigurdp):

  • Support metadata through .csv (in addition to .arrow and from .UNSMRY). We can define how that metadata should come together with the .csv.

@jorgenherje
Copy link
Collaborator

jorgenherje commented Jul 9, 2021

Above mentioned summary as checklist (for @jorgenherje):

Related factory responsibility (@sigurdp):

  • Support metadata through .csv (in addition to .arrow and from .UNSMRY). We can define how that metadata should come together with the .csv.

@jorgenherje
Copy link
Collaborator

VincentNevermore pushed a commit to VincentNevermore/webviz-subsurface that referenced this issue Jul 19, 2022
@anders-kiaer anders-kiaer moved this to Done 🏁 in Webviz Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants