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

Increase coverage by tiny amount #7

Open
wants to merge 6 commits into
base: main-test
Choose a base branch
from
Open

Conversation

lbianchi-lbl
Copy link
Owner

Fixes

Summary/Motivation:

Changes proposed in this PR:

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@lbianchi-lbl lbianchi-lbl marked this pull request as draft January 20, 2021 01:46
@lbianchi-lbl lbianchi-lbl marked this pull request as ready for review January 20, 2021 01:47
@lbianchi-lbl lbianchi-lbl requested a review from fndari January 20, 2021 01:49
@codecov-io
Copy link

codecov-io commented Jan 20, 2021

Codecov Report

Merging #7 (c130b27) into main-test (bd5213b) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           main-test       #7      +/-   ##
=============================================
- Coverage      64.67%   64.65%   -0.02%     
=============================================
  Files            274      274              
  Lines          41835    41835              
  Branches        7808     7808              
=============================================
- Hits           27057    27049       -8     
- Misses         13374    13378       +4     
- Partials        1404     1408       +4     
Impacted Files Coverage Δ
idaes/conftest.py 80.00% <0.00%> (-20.00%) ⬇️
idaes/dmf/codesearch.py 98.55% <0.00%> (-1.45%) ⬇️
idaes/dmf/util.py 69.68% <0.00%> (-1.36%) ⬇️
idaes/surrogate/pysmo/kriging.py 80.49% <0.00%> (-0.71%) ⬇️
idaes/ver.py 61.53% <0.00%> (+1.53%) ⬆️

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 0111cf9...62dc3c8. Read the comment docs.

@lbianchi-lbl lbianchi-lbl requested review from fndari, idaes-build and idaes-debug and removed request for fndari and idaes-build January 20, 2021 03:11
@lbianchi-lbl lbianchi-lbl requested review from idaes-build and removed request for idaes-build January 20, 2021 04:21
@lbianchi-lbl lbianchi-lbl added the CI:run-integration Trigger integration tests to run on this PR label Jan 20, 2021
@lbianchi-lbl lbianchi-lbl added approved CI:run-integration Trigger integration tests to run on this PR and removed CI:run-integration Trigger integration tests to run on this PR approved labels Jan 20, 2021
@lbianchi-lbl lbianchi-lbl added CI:run-integration Trigger integration tests to run on this PR and removed CI:run-integration Trigger integration tests to run on this PR approved labels Jan 20, 2021
@idaes-build idaes-build removed the CI:run-integration Trigger integration tests to run on this PR label Jan 20, 2021
@lbianchi-lbl lbianchi-lbl added CI-test-label CI:run-integration Trigger integration tests to run on this PR and removed CI-test-label CI:run-integration Trigger integration tests to run on this PR labels Jan 20, 2021
@lbianchi-lbl lbianchi-lbl added CI-test-label CI:run-integration Trigger integration tests to run on this PR added-by-bot and removed CI-test-label CI:run-integration Trigger integration tests to run on this PR labels Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
added-by-bot CI:run-integration Trigger integration tests to run on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants