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

Replace ecl with resdata #234

Merged
merged 4 commits into from
Nov 15, 2023
Merged

Replace ecl with resdata #234

merged 4 commits into from
Nov 15, 2023

Conversation

DanSava
Copy link
Contributor

@DanSava DanSava commented Nov 10, 2023

@codecov-commenter
Copy link

codecov-commenter commented Nov 10, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (69e7dd2) 83.31% compared to head (9a92224) 83.31%.

❗ Current head 9a92224 differs from pull request most recent head 3a22bdd. Consider uploading reports for the commit 3a22bdd to get more accurate results

Files Patch % Lines
src/fmu/ensemble/ensemble.py 80.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #234   +/-   ##
=======================================
  Coverage   83.31%   83.31%           
=======================================
  Files          14       14           
  Lines        3026     3026           
=======================================
  Hits         2521     2521           
  Misses        505      505           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DanSava
Copy link
Contributor Author

DanSava commented Nov 14, 2023

@berland Any idea why the tests are not starting? does this PR need to be verified by an admin? I tried making a PR from a branch created on the main repo but I got the same result.

@DanSava DanSava self-assigned this Nov 14, 2023
@berland
Copy link
Collaborator

berland commented Nov 14, 2023

The workflows were disabled due to inactivity. Now enabled again.

@DanSava DanSava force-pushed the replace_ecl branch 2 times, most recently from 4614693 to 1ee15c4 Compare November 14, 2023 09:15
Copy link

@andreas-el andreas-el left a comment

Choose a reason for hiding this comment

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

Looks good!
I think the -b html is by default, but I guess we can be explicit regarding this nonetheless.

@DanSava DanSava merged commit 459ddb0 into equinor:master Nov 15, 2023
4 checks passed
@DanSava DanSava deleted the replace_ecl branch November 15, 2023 13:36
DanSava added a commit that referenced this pull request Nov 22, 2023
* Replace ecl with resdata

* Fix doc yaml block

* Lock setuptools_scm version

* Fix doc building

(cherry picked from commit 459ddb0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants