Skip to content

Commit

Permalink
Merge pull request #60 from swagnercarena/fix_coveralls
Browse files Browse the repository at this point in the history
try to fix omit use in coverage.
  • Loading branch information
swagnercarena committed Jul 9, 2024
2 parents 7cd879b + 4fa6e8c commit 3a51d30
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,10 @@ jobs:
- name: Test with pytest
run: |
cd test
coverage run --source paltas --omit="*paltas/Configs/paper_2203_00690/*" -m unittest -v *_tests.py
coverage run --source paltas --omit "**/paltas/Configs/paper_2203_00690/**" -m unittest -v *_tests.py
coverage lcov
- name: Coveralls
uses: coverallsapp/github-action@master
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
path-to-lcov: ./test/coverage.lcov
path-to-lcov: ./test/coverage.lcov
2 changes: 1 addition & 1 deletion test/analysis_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -1063,7 +1063,7 @@ def test_build_xresnet34(self):
if 'dense' in layer.name:
self.assertFalse(layer.bias is None)
if 'stack4_block3_out' in layer.name:
self.assertListEqual(layer.output.shape.as_list(),
self.assertListEqual(list(layer.output.shape),
[None,2,2,512])
self.assertTupleEqual((None,num_outputs),model.output_shape)

Expand Down

0 comments on commit 3a51d30

Please sign in to comment.