-
-
Notifications
You must be signed in to change notification settings - Fork 203
Conversation
pablogsal
commented
Feb 8, 2022
I'm happy landing this. Let's give some days if @vstinner wants to take a look.
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
vstinner
commented
Feb 8, 2022
Does it belong to pyperformance which is supposed to be a set of "macro benchmarks"?
IMO it's to have benchmarks outside pyperformance. @ericsnowcurrently wants to add the ability to plug your own benchmark suite in pyperformance using configuration files, but I didn't follow his PR recently.
pablogsal
commented
Feb 8, 2022
Does it belong to pyperformance which is supposed to be a set of "macro benchmarks"?
I would say yes: we have things that are quite small in what they do anyway. C'mon, we even have unpacking tests:
pyperformance/pyperformance/data-files/benchmarks/bm_unpack_sequence/run_benchmark.py
Lines 10 to 20 in 034f58b
vstinner
commented
Feb 8, 2022
C'mon, we even have unpacking tests:
Maybe these tests don't belong to pyperformance :-(
I would say yes: we have things that are quite small in what they do anyway.
It's up to you.
ericsnowcurrently
commented
Feb 8, 2022
Does it belong to pyperformance which is supposed to be a set of "macro benchmarks"?
@ericsnowcurrently wants to add the ability to plug your own benchmark suite in pyperformance
My PR landed a while back and is part of the most recent release.
That said, we already have a bunch of micro benchmarks so I'm not opposed to adding more. (We can separate the bunch later.) However, it would be worth marking them as "micro" sooner rather than later so they are easy to skip. That would involve adding/updating tags = "micro" in the [tool.pyperformance] section of each benchmark's pyproject.toml. For this PR it would probably be okay to do so just for the new benchmark.
pablogsal
commented
Feb 9, 2022
@tiran THe benchmark fails for pypy because sys.getsizeof doesn't work. Check the CI output
Co-authored-by: Stan Ulbrych <stan@python.org>
No description provided.