| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Officially EOL by Python project Run `poetry update` Drop "backports.zoneinfo", importlib-resources Bump time-machine, pytest-codspeed to last version which support Py3.13 https://github.com/adamchainz/time-machine/blob/main/CHANGELOG.rst https://github.com/CodSpeedHQ/pytest-codspeed/releases
There was a problem hiding this comment.
One issue for now, let's see what CI says
Sorry, something went wrong.
CodSpeed Performance ReportMerging #863 will degrade performances by 91.71%Comparing M5oul:master (73cfa18) with master (09d815a) 🎉 Hooray! pytest-codspeed just leveled up to 3.0.0!
Summary❌ 1 regressions
Benchmarks breakdown
|
Sorry, something went wrong.
Directly import `zoneinfo` and `importlib.resources` in the files where used
|
I am no expert on Rust/PyO3, but are the changes to the code needed? There is a huge performance regression. It's unmergable in that state. |
Sorry, something went wrong.
|
I cherry-picked last commit from https://github.com/dlt-hub/pendulum/, otherwise, we get rust-lint errors that can be found bellow. I can’t say if the Rust changes are any good. error: non-local `impl` definition, `impl` blocks should be written at the same level as their item
--> src/python/types/duration.rs:23:1
|
23 | #[pymethods]
| -^^^^^^^^^^^
| |
| `PyClassImplCollector` is not local
| `PyClassNewTextSignature` is not local
| move the `impl` block outside of this function `trampoline` and up 4 bodies
24 | impl Duration {
| --------
| |
| `Duration` is not local
| `Duration` is not local
|
= note: the attribute macro `pymethods` defines the non-local `impl`, and may need to be changed
= note: the attribute macro `pymethods` may come from an old version of the `pyo3_macros` crate, try updating your dependency with `cargo update -p pyo3_macros`
= note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
= note: `-D non-local-definitions` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(non_local_definitions)]`
= note: this error originates in the attribute macro `pymethods` (in Nightly builds, run with -Z macro-backtrace for more info)
error: non-local `impl` definition, `impl` blocks should be written at the same level as their item
--> src/python/types/precise_diff.rs:23:1
|
23 | #[pymethods]
| -^^^^^^^^^^^
| |
| `PyClassImplCollector` is not local
| `PyClassNewTextSignature` is not local
| move the `impl` block outside of this function `trampoline` and up 4 bodies
24 | impl PreciseDiff {
| -----------
| |
| `PreciseDiff` is not local
| `PreciseDiff` is not local
|
= note: the attribute macro `pymethods` defines the non-local `impl`, and may need to be changed
= note: the attribute macro `pymethods` may come from an old version of the `pyo3_macros` crate, try updating your dependency with `cargo update -p pyo3_macros`
= note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
= note: this error originates in the attribute macro `pymethods` (in Nightly builds, run with -Z macro-backtrace for more info)
error: non-local `impl` definition, `impl` blocks should be written at the same level as their item
--> src/python/types/timezone.rs:11:1
|
11 | #[pymethods]
| -^^^^^^^^^^^
| |
| `PyClassImplCollector` is not local
| `PyClassNewTextSignature` is not local
| move the `impl` block outside of this function `trampoline` and up 4 bodies
12 | impl FixedTimezone {
| -------------
| |
| `FixedTimezone` is not local
| `FixedTimezone` is not local
|
= note: the attribute macro `pymethods` defines the non-local `impl`, and may need to be changed
= note: the attribute macro `pymethods` may come from an old version of the `pyo3_macros` crate, try updating your dependency with `cargo update -p pyo3_macros`
= note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
= note: this error originates in the attribute macro `pymethods` (in Nightly builds, run with -Z macro-backtrace for more info)
error: could not compile `_pendulum` (lib test) due to 3 previous errors
make: *** [Makefile:13: lint-rust] Error 101
codspeed performances regression seems related to its version bump, not the Pyo3/rust commit. I don’t have a significant difference with or without this commit: ┏━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━┓ ┃ Benchmark ┃ Time (best) ┃ Rel. StdDev ┃ Run time ┃ Iters ┃ with commit ┡━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━┩ │ test_parse_iso8601 │ 103,137ns │ 3.1% │ 2.91s │ 26,072 │ └────────────────────┴─────────────┴─────────────┴──────────┴────────┘ ┏━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━┓ ┃ Benchmark ┃ Time (best) ┃ Rel. StdDev ┃ Run time ┃ Iters ┃ without commit ┡━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━┩ │ test_parse_iso8601 │ 101,767ns │ 5.0% │ 3.00s │ 27,215 │ └────────────────────┴─────────────┴─────────────┴──────────┴────────┘ |
Sorry, something went wrong.
|
As said in codspeed message:
I gave it a try to compare on Py3.12, with only bumping pytest-codspeed version, based on master branch:
┏━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━┓ ┃ Benchmark ┃ Time (best) ┃ Rel. StdDev ┃ Run time ┃ Iters ┃ ┡━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━┩ │ test_parse_iso8601 │ 147,445ns │ 4.4% │ 3.00s │ 18,919 │ └────────────────────┴─────────────┴─────────────┴──────────┴────────┘ I don’t have this table with previous versions, not sure which value should be taken. |
Sorry, something went wrong.
|
I will try to find some time to test the changes myself. I just want to be extra careful and double-check everything. |
Sorry, something went wrong.
|
@M5oul How can I solve problem about rust impl? |
Sorry, something went wrong.
|
Ok, I have tested the changes locally and it seems to be alright, gonna merge it and see how it behaves with some PRs I plan on merging soon-ish. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
at least from the dependencies point of view.
Bumping dependencies, which dropped Py3.8 supports, also adds Py3.13 support.
(#844)