Repository navigation
test: add benchmark tests for encode/decode operations - #56
sumanjeet0012 wants to merge 3 commits into
Conversation
acul71
left a comment
There was a problem hiding this comment.
Review (maintainer)
Thanks for the clean benchmark addition — it matches #42 and the go-multibase encode/decode/roundtrip shape, and local/CI validation looks good once pytest-benchmark is present.
Blocking
Missing newsfragment — this is a mandatory requirement for approval.
Please add something like:
newsfragments/42.misc.rst(or42.internal.rst)
with a short ReST line describing the change for HISTORY, ending with a trailing newline. Example:
Added pytest-benchmark encode/decode/roundtrip benchmarks covering all multibase encodings.Non-blocking but important
- Branch is 6 commits behind
master— please rebase (or merge) onto currentorigin/masterbefore landing. - CI cost: these benchmarks run in the normal pytest suite (~40s+ locally for 72 timed cases). Consider default-disabling them in regular CI (
--benchmark-disable/ config) and documenting how to run them explicitly, unless we intentionally want always-on timings. pyproject.tomloverlap with open PRs #49 and #57 — informational only; expect a trivial dependency-list conflict when those merge.
Full structured review saved locally as AI-PR-REVIEW-56-0.md.
|
Hi @sumanjeet0012 — friendly follow-up. I requested changes on this PR on 2026-09-07 (mainly the missing towncrier newsfragment, plus the other notes in that review). I don’t see any new commits addressing those items yet. Are you still willing / able to carry this PR forward? If yes, please push the requested updates when you can and I’ll re-review promptly. If your bandwidth has changed, no worries — just say so and we can close it or leave it for someone else to pick up. |
🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
Fixes #42
Description
This pull request adds benchmark tests for all encodings to establish performance baselines and help catch regressions. Similar to the benchmarks in
go-multibase(BenchmarkEncode,BenchmarkDecode,BenchmarkRoundTrip), these benchmark tests run across all supported implementations.Changes
pytest-benchmarkas a development dependency inpyproject.toml.tests/test_benchmarks.pycontainingtest_bench_encode,test_bench_decode, andtest_bench_roundtripparameterized across allENCODINGS.