Sparse Ops Cleanup #19782
Sparse Ops Cleanup #19782
Conversation
| is_self_int_dtype = is_integer_dtype(self.dtype) | ||
| | ||
| fill_int = lambda x: x.fillna(0) | ||
| fill_bool = lambda x: x.fillna(False).astype(bool) |
There was a problem hiding this comment.
Moving these outside of the function scope to clarify they don't need to be re-defined on each call. _bool_method_SERIES is the topic of a separate PR, but this is sufficiently small to include in assorted-cleanups.
Codecov Report
@@ Coverage Diff @@ ## master #19782 +/- ## ========================================== + Coverage 91.61% 91.61% +<.01% ========================================== Files 150 150 Lines 48887 48882 -5 ========================================== - Hits 44786 44782 -4 + Misses 4101 4100 -1
Continue to review full report at Codecov.
|
| other = SparseArray(other, fill_value=self.fill_value, | ||
| dtype=dtype) | ||
| return _sparse_array_op(self, other, op, name) | ||
| return _sparse_array_op(self, other, op, name, series=False) |
There was a problem hiding this comment.
this whole sparse interaction is pretty hacky (I know not addressing this now), but....
There was a problem hiding this comment.
Yah, there are a couple of improvements in store for the pass after next (which is just implementing the single-call ops.add_whatever_methods(cls))
| sparse_series_special_funcs = dict(arith_method=_arith_method_SPARSE_SERIES, | ||
| comp_method=_arith_method_SPARSE_SERIES, | ||
| bool_method=None) | ||
| bool_method=_bool_method_SERIES) |
There was a problem hiding this comment.
can we hold off adding till then
There was a problem hiding this comment.
This is necessary to get rid of the double-call in sparse.series.
| thanks |
2 participants
This is in preparation for implementing the direct-pinning discussed here. The main roadblock to that is SparseSeries which calls ops.add_special_arithmetic_methods twice. This PR gets rid of the double-call, so the follow-up will be straight-forward.
A handful of cleanups are done here too:
SparseArray.__eq__.__name__ == 'eq'