Skip to content
This repository was archived by the owner on Feb 2, 2024. It is now read-only.

Commit e663d33

Browse files
committed
Minor change in 1 test on dataframe
1 parent 9d115fd commit e663d33

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

sdc/hiframes/boxing.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,8 @@ def box_dataframe(typ, val, c):
247247
# context.nrt.incref(builder, arr_typ, arr)
248248
pyapi.dict_setitem(df_dict, cname_obj, arr_obj)
249249

250-
c.context.nrt.decref(c.builder, list_type, typ_arrs.value)
250+
if c.context.enable_nrt:
251+
c.context.nrt.decref(c.builder, list_type, list_val)
251252
pyapi.decref(arr_obj)
252253
pyapi.decref(cname_obj)
253254

sdc/tests/test_dataframe.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2333,15 +2333,19 @@ def test_append_df_diff_cols_index_ignore_false_no_unboxing(self):
23332333
def test_impl():
23342334
n1 = 11
23352335
n2 = n1 * 2
2336-
df = pd.DataFrame({'A': np.arange(n1), 'B': np.arange(n1) ** 2},
2337-
index=np.arange(n1) ** 2)
2338-
df2 = pd.DataFrame({'C': np.arange(n2), 'D': np.arange(n2) ** 2,
2339-
'E S D': np.arange(n2) + 100},
2340-
index=np.arange(n2) ** 4)
2336+
df = pd.DataFrame({
2337+
'A': np.arange(n1), 'B': np.arange(n1) ** 2
2338+
}, index=np.arange(n1) ** 2)
2339+
df2 = pd.DataFrame({
2340+
'C': np.arange(n2), 'D': np.arange(n2) ** 2,
2341+
'E S D': np.arange(n2) + 100
2342+
}, index=np.arange(n2) ** 4)
23412343
return df.append(df2, ignore_index=False)
23422344

23432345
sdc_func = self.jit(test_impl)
2344-
pd.testing.assert_frame_equal(sdc_func(), test_impl())
2346+
res_jit = sdc_func()
2347+
res_ref = test_impl()
2348+
pd.testing.assert_frame_equal(res_jit, res_ref)
23452349

23462350
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
23472351
def test_append_df_diff_cols_index_ignore_index(self):

0 commit comments

Comments
 (0)