fixup! ENH: SparseDataFrame supports scipy.sparse.spmatrix in setitem

Author: kernc

pandas/core/sparse/frame.py

@@ -436,11 +436,12 @@ def __getitem__(self, key):
     def __setitem__(self, key, value):
         if is_scipy_sparse(value):
             if any(ax == 1 for ax in value.shape):  # 1d spmatrix
-                value = SparseArray(value, fill_value=self._default_fill_value)
+                value = SparseArray(value, fill_value=self._default_fill_value,
+                                    kind=self._default_kind)
             else:
                 # 2d; make it iterable
                 value = list(value.tocsc().T)
-        super().__setitem__(key, value)
+        super(SparseDataFrame, self).__setitem__(key, value)
 
     @Appender(DataFrame.get_value.__doc__, indents=0)
     def get_value(self, index, col, takeable=False):

pandas/tests/sparse/test_frame.py

@@ -557,8 +557,9 @@ def test_setitem_spmatrix(self):
         assert (sdf[['X']].to_coo() != spm.tocoo()).nnz == 0
 
         # 1d row -- changing series contents not yet supported
-        spm = csr_matrix(np.arange(sdf.shape[1])).astype(float)
-        idx = np.r_[[False, True], np.full(sdf.shape[0] - 2, False)]
+        spm = csr_matrix(np.arange(sdf.shape[1], dtype=float))
+        idx = np.zeros(sdf.shape[0], dtype=bool)
+        idx[1] = True
         tm.assert_raises_regex(TypeError, 'assignment',
                                lambda: sdf.__setitem__(idx, spm))