Closed benedekrozemberczki closed 2 years ago
@benedekrozemberczki flake8 got you!
I know.
Merging #43 (15e4516) into main (53ca94b) will decrease coverage by
0.01%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #43 +/- ##
==========================================
- Coverage 98.65% 98.64% -0.02%
==========================================
Files 28 27 -1
Lines 671 664 -7
==========================================
- Hits 662 655 -7
Misses 9 9
Impacted Files | Coverage Δ | |
---|---|---|
chemicalx/__init__.py | 100.00% <ø> (ø) |
|
chemicalx/models/__init__.py | 100.00% <ø> (ø) |
|
tests/unit/test_models.py | 100.00% <ø> (ø) |
|
chemicalx/models/deepsynergy.py | 100.00% <100.00%> (ø) |
|
chemicalx/models/epgcnds.py | 100.00% <100.00%> (ø) |
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 53ca94b...15e4516. Read the comment docs.
Summary
Removes the DeepCCI class.
Adds Out channels for DeepSynergy and EPGCN DS.
Moves EPGCN-DS to GraphConv layers.
[ ] Code passes all tests
[x] Unit tests provided for these changes
[x] Documentation and docstrings added for these changes