-
-
Notifications
You must be signed in to change notification settings - Fork 144
[fix] No overload variant of "isna" matches argument type "NAType" [call-overload] #279
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…all-overload] Update `isna` & `notna` overloads inside pandas-stubs/core/dtypes/missing.pyi Update `test_isna` function inside tests/test_pandas.py file Refs #264
When I tried to test with: check(assert_type(pd.isna(pd.NA), bool), bool)
check(assert_type(pd.notna(pd.NA), bool), bool) I had the following errors:
I decided to use check(assert_type(pd.isna(np.nan), bool), bool)
check(assert_type(pd.notna(np.nan), bool), bool) |
I think you pulled the wrong type. You need to use >>> import pandas as pd
>>> type(pd.NA)
<class 'pandas._libs.missing.NAType'>
>>> type(pd.NaT)
<class 'pandas._libs.tslibs.nattype.NaTType'> |
…all-overload] Update `isna` & `notna` overloads inside pandas-stubs/core/dtypes/missing.pyi Update `test_isna` function inside tests/test_pandas.py file Refs #264
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.
This all looks good except for one small thing.
…all-overload] Update `isna` & `notna` overloads inside pandas-stubs/core/dtypes/missing.pyi Update `test_isna` function inside tests/test_pandas.py file Refs #264
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.
hit the wrong button - make the changes with the tests and we should be good
…all-overload] Update `isna` & `notna` overloads inside pandas-stubs/core/dtypes/missing.pyi Update `test_isna` function inside tests/test_pandas.py file Refs #264
…all-overload] Update `isna` & `notna` overloads inside pandas-stubs/core/dtypes/missing.pyi Update `test_isna` function inside tests/test_pandas.py file Refs #264
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.
Thanks @LeMeteore
Thanks to you all o/ I've learned something today. |
re this one, I'm getting errors in mypy due |
Can you create an issue, with the example, and the output of |
here you go #318 |
Update
isna
¬na
overloads inside pandas-stubs/core/dtypes/missing.pyiUpdate
test_isna
function inside tests/test_pandas.py fileRefs error: No overload variant of "isna" matches argument type "NAType" [call-overload] #264
Tests updated