Closed kmdalton closed 3 years ago
Merging #76 (d6339d2) into main (6e0c932) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #76 +/- ##
=======================================
Coverage 98.92% 98.92%
=======================================
Files 40 40
Lines 1488 1494 +6
=======================================
+ Hits 1472 1478 +6
Misses 16 16
Flag | Coverage Δ | |
---|---|---|
unittests | 98.92% <100.00%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
reciprocalspaceship/dataset.py | 99.07% <100.00%> (+0.01%) |
:arrow_up: |
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 712a6bb...d6339d2. Read the comment docs.
I added a method to
rs.DataSet
which just drops systematically absent rows. This saves the user having to do something likewhich can now by done by
. This method supports the
inplace
argument, and by usingpd.DataFrame.drop
and operating at theds.index
level, I think it avoids using extra memory unnecessarily. This will be helpful for large, unmerged datasets. The recipe was a tad tricky to work out, so I figured we should just put it in a method. (Admittedly this part is a premature optimization, but I do think the method makes code cleaner at the user level).@JBGreisman , let me know what you think