search

pydata / patsy

Describing statistical models in Python using symbolic formulas
Other
947 stars 103 forks source link

Add support for Python 3.12 (and adjust code for tokenizer changes). #197

Closed matthewwardrop closed 11 months ago

matthewwardrop commented 11 months ago

closes: #196

bashtage commented 11 months ago

Is there any expectations the CI runs or needs to be passed?

matthewwardrop commented 11 months ago

Is there any expectations the CI runs or needs to be passed?

Yes... but it is struggling because Python 2.7 and 3.6 no longer exist in GitHub actions support versions. Fixing.

codecov-commenter commented 11 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (6bd31ca) 98.49% compared to head (8853aee) 98.43%.

Files Patch % Lines
patsy/tokens.py 60.00% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #197 +/- ## ========================================== - Coverage 98.49% 98.43% -0.07% ========================================== Files 30 30 Lines 3131 3132 +1 Branches 690 691 +1 ========================================== - Hits 3084 3083 -1 - Misses 24 25 +1 - Partials 23 24 +1 ``` | [Flag](https://app.codecov.io/gh/pydata/patsy/pull/197/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pydata) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/pydata/patsy/pull/197/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pydata) | `98.02% <60.00%> (-0.07%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pydata#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.