Closed rkurchin closed 3 years ago
@SeanXiaoyuSun presuming this CI passes, could you review this when you have a minute? I need it for testing the package on QM9...
Merging #21 into master will decrease coverage by
1.25%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #21 +/- ##
==========================================
- Coverage 84.03% 82.77% -1.26%
==========================================
Files 6 6
Lines 357 360 +3
==========================================
- Hits 300 298 -2
- Misses 57 62 +5
Impacted Files | Coverage Δ | |
---|---|---|
src/ChemistryFeaturization.jl | 100.00% <ø> (ø) |
|
src/atomfeat.jl | 92.17% <100.00%> (ø) |
|
src/pmg_graphs.jl | 83.54% <100.00%> (-5.93%) |
:arrow_down: |
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 41bf2f1...5238c3b. Read the comment docs.
Converting over to using ASE's
neighbor_list
rather than the pymatgen version, which ends up much cleaner anyway.