-
Notifications
You must be signed in to change notification settings - Fork 61
Fixing issue with named series handling in fillna #95
Fixing issue with named series handling in fillna #95
Conversation
@@ -361,7 +361,7 @@ def generic(self, args, kws): | |||
class FillNaStrType(AbstractTemplate): | |||
def generic(self, args, kws): | |||
assert not kws | |||
assert len(args) == 2 | |||
assert len(args) == 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is necessary to correct the comment under this line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's done.
'expecting {{i64, i64, i32*, i8*, i8*, i8*}, i8*, ' | ||
'{i8*, i64, i32, i32, i64, i8*, i8*}} \n' | ||
'but got {{i64, i64, i32*, i8*, i8*, i8*}, i8*, i8*}\n') | ||
def test_series_fillna_inplace1(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a comment what this function is testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added some comment (there are no comments for other tests though).
This new test just adds missed check for attempting to jit inplace fillna on series with numeric data. There was a test for string series, but not numeric one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fine. We want to improve code style and docu. So anything new should be well documented.
a60740a
to
7ce6277
Compare
[BUG] Fixed problems with generation parquet files (IntelPython#93)
7ce6277
to
065c970
Compare
Fixing issue with named series handling in fillna (IntelPython#95)
No description provided.