-
Notifications
You must be signed in to change notification settings - Fork 104
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
Add tests for save/load parameters with rowscaling #2721
Add tests for save/load parameters with rowscaling #2721
Conversation
4cbd02f
to
52a5dfa
Compare
Codecov Report
@@ Coverage Diff @@
## main #2721 +/- ##
==========================================
+ Coverage 65.23% 65.28% +0.04%
==========================================
Files 651 651
Lines 53549 53556 +7
Branches 4784 4786 +2
==========================================
+ Hits 34935 34964 +29
+ Misses 17041 16988 -53
- Partials 1573 1604 +31
Continue to review full report at Codecov.
|
ministep, row_scaling_list); | ||
|
||
WHEN("loading parameters from enkf_fs") { | ||
auto parmeter_matrices = analysis::load_row_scaling_parameters( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
parameter_matrices
?
Pedantizm 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is potentially a list of matrices 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking that parmeter
should be parameter
...
No biggie 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙈 I will fix
54fcd99
to
5d3d51b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
5d3d51b
to
d1a32f8
Compare
d1a32f8
to
12cc1d1
Compare
Test this please! |
3 similar comments
Test this please! |
Test this please! |
Test this please! |
Issue
Resolves #2582
Resolves #2584
Approach
Roundtrip test for a matrix with rowscaling attached
Pre review checklist
Adding labels helps the maintainers when writing release notes, see sections and the
corresponding labels here: https://github.com/equinor/ert/blob/main/.github/release.yml