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

Re-implement DataFrame unboxing #860

Merged
densmirn merged 13 commits into from
May 29, 2020
Merged

Re-implement DataFrame unboxing #860

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
4e5cd9a
Boxing draft
akharche May 25, 2020
06e9295
Merge branch 'feature/dataframe_model_refactoring' of https://github.…
akharche May 27, 2020
ce499ff
Implement unboxing in new structure
akharche May 27, 2020
fdf672e
Improve variable names + add error handling
akharche May 28, 2020
f164c3d
Return error status
akharche May 28, 2020
99e9269
Move getting list size to if_ok block
akharche May 28, 2020
ce771b8
Unskipped unexpected success tests
akharche May 28, 2020
6b45a77
Unskipped unexpected success tests in GroupBy
akharche May 28, 2020
0814cc3
Merge branch 'feature/dataframe_model_refactoring' of https://github.…
akharche May 29, 2020
66b40a8
Remove decorators
akharche May 29, 2020
1d43ce6
Change to incref False
akharche May 29, 2020
8ee9364
Skip tests failed due to unimplemented df structure
akharche May 29, 2020
666fceb
Bug in rolling
akharche May 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Unskipped unexpected success tests in GroupBy
  • Loading branch information
@akharche
akharche committed May 28, 2020
commit 6b45a77dd1af0c604feb16f589fc1ad1bb32e77b
11 changes: 0 additions & 11 deletions sdc/tests/test_groupby.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -447,7 +447,6 @@ def test_impl(df):
pd.testing.assert_frame_equal(result, result_ref, check_names=False)

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_count(self):
def test_impl(df):
return df.groupby('A')['B'].count()
Expand All @@ -459,7 +458,6 @@ def test_impl(df):
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_mean(self):
def test_impl(df):
return df.groupby('A')['B'].mean()
Expand All @@ -471,7 +469,6 @@ def test_impl(df):
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_median(self):
def test_impl(df):
return df.groupby('A')['B'].median()
Expand All @@ -483,7 +480,6 @@ def test_impl(df):
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_min(self):
def test_impl(df):
return df.groupby('A')['B'].min()
Expand All @@ -505,7 +501,6 @@ def test_impl(df):
self.assertEqual(set(hpat_func(df)), set(test_impl(df)))

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_max(self):
def test_impl(df):
return df.groupby('A')['B'].max()
Expand All @@ -527,7 +522,6 @@ def test_impl(df):
self.assertEqual(set(hpat_func(df)), set(test_impl(df)))

@skip_sdc_jit("Old-style implementation returns ndarray, not a Series")
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_prod(self):
def test_impl(df):
return df.groupby('A')['B'].prod()
Expand All @@ -539,7 +533,6 @@ def test_impl(df):
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@skip_sdc_jit
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_var(self):
def test_impl(df):
return df.groupby('A')['B'].var()
Expand All @@ -551,7 +544,6 @@ def test_impl(df):
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@skip_sdc_jit
@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_agg_seq_std(self):
def test_impl(df):
return df.groupby('A')['B'].std()
Expand Down Expand Up @@ -854,7 +846,6 @@ def test_impl(df):
# TODO: implement index classes, as current indexes do not have names
pd.testing.assert_frame_equal(result, result_ref, check_names=False)

@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_dataframe_groupby_getitem_literal_str(self):
def test_impl(df):
return df.groupby('C')['B'].count()
Expand All @@ -866,7 +857,6 @@ def test_impl(df):
# TODO: implement index classes, as current indexes do not have names
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_dataframe_groupby_getitem_unicode_str(self):
def test_impl(df, col_name):
return df.groupby('A')[col_name].count()
Expand All @@ -880,7 +870,6 @@ def test_impl(df, col_name):
# TODO: implement index classes, as current indexes do not have names
pd.testing.assert_series_equal(result, result_ref, check_names=False)

@dfRefactoringNotImplemented # required re-implementing DataFrame unboxing
def test_dataframe_groupby_getitem_repeated(self):
def test_impl(df):
return df.groupby('A')['B', 'C']['D']
Expand Down