-
-
Notifications
You must be signed in to change notification settings - Fork 141
CLEAN: Reconcile types in pandas._typing.py and pandas-stubs._typing.pyi #128
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
Comments
Are there any tools that will help keep stubs in sync with pandas? When I went through io/stata.py there were a lot of changes since these were originally collatted. |
Not that I'm aware of. Maybe |
Might not make sense to fix all of them (some exist on main but not in 1.4.x) but it gives a quick overview what could be changed
|
Can this tell us which ones are in Also would be best if we could use what is in |
I think so, would probably need to remove
stubtest will use whichever version of pandas that is installed. Currently, that is 1.4.x. I think pandas has nightly builds, we could install that in pyproject.toml instead of 1.4.x: but then we might get some "troubles" with pytests, more deprecation warnings (before 1.5) and breaking changes (after 1.5 is released). |
I wrote some quick&ugly ast parsing code to check when pandas-stubs has annotations but pandas has none (I have been using it for pandas-dev/pandas#47827 and similar PRs). Could extend that, but stubtest without |
Maybe you could just figure out a way to run stubtest (but not mypy, pytest, etc.) against the current version in |
Could have a step ala "remove pandas 1.4x. and install pandas nightly" with an appropriate rollback. |
There are some inconsistencies in types in
pandas._typing.py
andpandas-stubs._typing.pyi
. These should be reconciled.In addition, types that are only in the stubs should probably be private with a leading underscore
_
The text was updated successfully, but these errors were encountered: