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

Allow mixing sensitivity and gen_kw ensembles in ParametersModel #855

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

tnatt
Copy link
Collaborator

@tnatt tnatt commented Nov 24, 2021

PR to support a mix of sensitivity ensembles and gen_kw ensembles in ParametersModel.
A dice is added as SENSNAME to gen_kw ensembles for easy distinction from sensitivity ensembles:
image

Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍 One comment below only from my point of view.

For my understanding: Are there use cases where it is valuable for the user not to allow mixing? If not, should it always be allowed and we can remove allow_ensemble_type_mix input argument?

webviz_subsurface/_models/parameter_model.py Outdated Show resolved Hide resolved
@anders-kiaer anders-kiaer added the next release 🚢 To be included in next release label Nov 25, 2021
@tnatt
Copy link
Collaborator Author

tnatt commented Nov 25, 2021

LGTM. 👍 One comment below only from my point of view.

For my understanding: Are there use cases where it is valuable for the user not to allow mixing? If not, should it always be allowed and we can remove allow_ensemble_type_mix input argument?

Originally I just removed the Error, but than I thought we might have plugins were it might be important to only have one.. but thinking about it again, as long as we provide sensitivity columns to non-sensitivity ensembles it should not be a problem to include them in any plugin :) I'll clean up the PR and remove that option.. can rather put it back in if ever needed 👍

@tnatt tnatt force-pushed the support-both-ensembletypes branch from 53b6d72 to 8560435 Compare November 25, 2021 09:49
Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎄

@tnatt tnatt merged commit 8fd9504 into equinor:master Nov 25, 2021
VincentNevermore pushed a commit to VincentNevermore/webviz-subsurface that referenced this pull request Jul 19, 2022
Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.7 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](unshiftio/url-parse@1.5.7...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Håvard Bjerke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release 🚢 To be included in next release
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants