Skip to content
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

Typing/mypy settings #1911

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

m-richards
Copy link
Collaborator

Supporting mypy changes to experiment with resolving mypy issues in #1905.

I've just initially pushed now with failing tests to show a point, but I'll probably drive discussion from the other PR for now.

Copy link

codecov bot commented Feb 15, 2025

Codecov Report

Attention: Patch coverage is 0% with 32 lines in your changes missing coverage. Please review.

Project coverage is 65.00%. Comparing base (812b2a8) to head (c8af059).
Report is 193 commits behind head on main.

Files with missing lines Patch % Lines
pandera/engines/polars_engine.py 0.00% 22 Missing ⚠️
pandera/api/pyspark/types.py 0.00% 3 Missing ⚠️
pandera/backends/polars/builtin_checks.py 0.00% 3 Missing ⚠️
pandera/api/pyspark/column_schema.py 0.00% 2 Missing ⚠️
pandera/api/pyspark/model.py 0.00% 1 Missing ⚠️
pandera/backends/pyspark/decorators.py 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (812b2a8) and HEAD (c8af059). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (812b2a8) HEAD (c8af059)
48 47
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1911       +/-   ##
===========================================
- Coverage   94.28%   65.00%   -29.28%     
===========================================
  Files          91      121       +30     
  Lines        7013     9374     +2361     
===========================================
- Hits         6612     6094      -518     
- Misses        401     3280     +2879     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

m-richards and others added 14 commits February 23, 2025 22:29
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
…nai-oss#1905)

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* trial type annotations

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* changes in individual api files

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* pl.dataframe working in local test

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* older python union compat

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* try polars in the mypy env on ci

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* translate toplevel mypy skip into module specific skips

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* mypy passes

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* missing line continuation

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* python 3.8

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

---------

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
* declared support for python 3.12

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* set python3.8 as min python in setup's 'python_requires'

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* bumped pylint to <3.3 (3.2.x latest to be run with 3.8)

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* added pyupgrade and applied modifications

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* .github/tests: skipped pyspark tests on 3.12

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* TEMP: disabled pylint warnings (possibly-used-before-assignment). need addressing

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* .github: added setuptools to test run deps (required by noxfile), since it's no longer installed by default with python 3.12

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* added setuptools as an explicit test dependency

it is required in noxfile and not automatically provided by python >= 3.12

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* updated dependency spec of typing_extensions

typing_extensions is currently required by all python versions

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

* updated autogen requirements files

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>

---------

Signed-off-by: Guillaume Andreu Sabater <guillaume.andreu.sabater@gmail.com>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
…nai-oss#1905)

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* trial type annotations

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* changes in individual api files

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* pl.dataframe working in local test

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* older python union compat

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* try polars in the mypy env on ci

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* translate toplevel mypy skip into module specific skips

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* mypy passes

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* missing line continuation

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

* python 3.8

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>

---------

Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Signed-off-by: Matt Richards <mrichards7@outlook.com.au>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants