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

Delete unused arg_pack.h #2595

Merged
merged 1 commit into from
Dec 15, 2021
Merged

Delete unused arg_pack.h #2595

merged 1 commit into from
Dec 15, 2021

Conversation

dafeda
Copy link
Contributor

@dafeda dafeda commented Dec 15, 2021

Issue
Resolves #2594

Pre review checklist

  • [ x] Added appropriate labels

@dafeda dafeda added the maintenance Not a bug now but could be one day, repaying technical debt label Dec 15, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 15, 2021

Codecov Report

Merging #2595 (e884fcc) into main (6a48a2c) will decrease coverage by 13.65%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #2595       +/-   ##
===========================================
- Coverage   64.52%   50.86%   -13.66%     
===========================================
  Files         646      160      -486     
  Lines       54500    24874    -29626     
  Branches     4516     4199      -317     
===========================================
- Hits        35164    12653    -22511     
+ Misses      17949    10825     -7124     
- Partials     1387     1396        +9     
Impacted Files Coverage Δ
libres/lib/include/ert/enkf/row_scaling.hpp 0.00% <0.00%> (-100.00%) ⬇️
libres/lib/include/ert/res_util/matrix.hpp 50.00% <0.00%> (-50.00%) ⬇️
libres/lib/enkf/value_export.cpp 58.49% <0.00%> (-15.24%) ⬇️
libres/lib/config/conf_util.cpp 55.55% <0.00%> (-14.60%) ⬇️
libres/lib/job_queue/forward_model.cpp 48.33% <0.00%> (-7.99%) ⬇️
libres/lib/analysis/ies/ies_enkf_data.cpp 83.00% <0.00%> (-7.06%) ⬇️
libres/lib/res_util/es_testdata.cpp 61.14% <0.00%> (-6.72%) ⬇️
libres/lib/job_queue/job_kw_definitions.cpp 60.00% <0.00%> (-6.67%) ⬇️
libres/lib/enkf/enkf_util.cpp 50.00% <0.00%> (-6.53%) ⬇️
libres/lib/enkf/enkf_main_ensemble.cpp 63.33% <0.00%> (-5.90%) ⬇️
... and 619 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a48a2c...e884fcc. Read the comment docs.

@eivindjahren
Copy link
Contributor

weirdly this seems to change the behavior of some ert_gui tests. May be worth having a look.

@dafeda
Copy link
Contributor Author

dafeda commented Dec 15, 2021

weirdly this seems to change the behavior of some ert_gui tests. May be worth having a look.

All tests have passed, which ones have changed behaviour?

@eivindjahren
Copy link
Contributor

eivindjahren commented Dec 15, 2021

All tests have passed, which ones have changed behaviour?

Not sure but coverage no longer hits ert_gui.ertwidgets.validationsupport::presentError. Might be a fluke. Might be that an error related to arg_pack.h no longer makes it way to the gui which I count as a good thing :)

@eivindjahren
Copy link
Contributor

eivindjahren commented Dec 15, 2021

Btw, the coverage changes I am looking at are here: https://app.codecov.io/gh/equinor/ert/compare/2595/changes

@dafeda dafeda self-assigned this Dec 15, 2021
@dafeda dafeda merged commit 079c0bd into equinor:main Dec 15, 2021
@dafeda dafeda deleted the remove_arg_pack branch December 15, 2021 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Not a bug now but could be one day, repaying technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove unused arg_pack.h
3 participants