[benchmark] Request for call-path / fairness review (GLD.SerializerBenchmark)
- Linguagem predominante
- Rust
- Estrelas
- 305
- Forks
- 79
- Merge médio
- 21h 59min
- PRs com merge (30d)
- 2
Descrição
Hello,
I'm Leonid Ganeline. We maintain an open multi-language serializer benchmark:
https://github.com/leo-gan/GLD.SerializerBenchmark
Your library is included in the suite (language: python, harness name: `cbor2`).
Implementation we use:
https://github.com/leo-gan/GLD.SerializerBenchmark/blob/master/python/src/benchmark/serializers/binary_cbor2.py
We would value a short review of whether our measurement is fair and idiomatic:
1. Does our call pattern match how you recommend using the library in performance-sensitive code?
2. Should we change options, encoder/decoder reuse, types, or buffer handling?
3. Is there a better API (or a second entry point worth a separate row in the suite)?
Concrete notes or a small PR against that wrapper would help a lot. We are happy to credit you in the docs.
The repo also includes a short Serialization course (101–401). If something important about your design is easy to misstate, a pointer is welcome—we can update the docs ourselves.
Thank you for maintaining this library.
Best regards,
Leonid Ganeline
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Direção de pesquisa
Start with the benchmark wrapper at python/src/benchmark/serializers/binary_cbor2.py and compare its call pattern with cbor2's documented usage. Review encoder and decoder reuse, options, types, and buffer handling, then report concrete fairness findings or prepare a small PR against that wrapper; any design details that affect the Serialization course docs can be noted separately.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python, rust
- Domínio
- performance, testing
- Tipo de issue
- Refatoração
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Ativa
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 48/100