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

ProdMisfit- fixed bug related to use of excl_name_contains. #1055

Merged
merged 4 commits into from
Jun 14, 2022

Conversation

rnyb
Copy link
Collaborator

@rnyb rnyb commented Jun 10, 2022

It was not taken into account, i.e. it had no effect, due to badly placed continue
inside nested loop.

It was not taken into account, i.e. it had no effect, due to badly placed continue
inside nested loop.
@rnyb rnyb added the bug 🐛 Something isn't working label Jun 10, 2022
@rnyb rnyb requested a review from anders-kiaer June 10, 2022 22:22
@rnyb rnyb changed the title Fixed bug related to use of excl_name_contains. ProdMisfit- fixed bug related to use of excl_name_contains. Jun 10, 2022
@rnyb rnyb self-assigned this Jun 10, 2022
@rnyb
Copy link
Collaborator Author

rnyb commented Jun 13, 2022

Closes #1056

@anders-kiaer anders-kiaer linked an issue Jun 14, 2022 that may be closed by this pull request
@rnyb rnyb merged commit 0f05a53 into equinor:master Jun 14, 2022
@rnyb rnyb deleted the prod-misfit-fix branch June 14, 2022 12:07
VincentNevermore pushed a commit to VincentNevermore/webviz-subsurface that referenced this pull request Jul 19, 2022
fix equinor#756

Co-authored-by: Shadab Khan <Shadab.Khan@Emerson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
Status: Done 🏁
Development

Successfully merging this pull request may close these issues.

Adding endswith filtering option to ProdMisfit
2 participants