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

Update CI #258

Merged
merged 6 commits into from
Jun 24, 2021
Merged

Update CI #258

merged 6 commits into from
Jun 24, 2021

Conversation

adrhill
Copy link
Collaborator

@adrhill adrhill commented Jun 24, 2021

This PR updates the GitHub Actions CI:

  • separates linting from the test matrix (OS / Python version)
  • fixes code coverage with codecov

Add matrix test with fast tests and coverage test with precommit flag.

Adapted from examples here:
* https://github.com/snok/install-poetry
* https://github.com/codecov/codecov-action
allowing for 3.8 and 3.9 to be used
due to unavailability of TF 1.15.3
this can later be relaxed again when moving away from TF1.15
@codecov-commenter
Copy link

Codecov Report

Merging #258 (2ac3538) into master (c3295be) will increase coverage by 49.94%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #258       +/-   ##
===========================================
+ Coverage   21.41%   71.36%   +49.94%     
===========================================
  Files          46       41        -5     
  Lines        4300     4173      -127     
  Branches      659      637       -22     
===========================================
+ Hits          921     2978     +2057     
+ Misses       3357     1007     -2350     
- Partials       22      188      +166     
Impacted Files Coverage Δ
src/innvestigate/analyzer/__init__.py
src/innvestigate/utils/keras/__init__.py
src/innvestigate/utils/__init__.py
src/innvestigate/tools/__init__.py
src/innvestigate/__init__.py
src/innvestigate/analyzer/relevance_based/utils.py 32.35% <0.00%> (+32.35%) ⬆️
src/innvestigate/utils/keras/backend.py 35.08% <0.00%> (+35.08%) ⬆️
src/innvestigate/applications/imagenet.py 37.83% <0.00%> (+37.83%) ⬆️
src/innvestigate/utils/keras/graph.py 56.02% <0.00%> (+56.02%) ⬆️
src/innvestigate/tools/perturbate.py 62.98% <0.00%> (+62.98%) ⬆️
... and 13 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 c3295be...2ac3538. Read the comment docs.

@adrhill adrhill requested a review from albermax June 24, 2021 15:07
@albermax albermax merged commit bcabb2e into albermax:master Jun 24, 2021
@adrhill adrhill deleted the adrhill/update-ci branch June 24, 2021 17:09
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.

None yet

3 participants