agronholm / agronholm/cbor2

[benchmark] Request for call-path / fairness review (GLD.SerializerBenchmark)

Aberta
#330 2 comentários 0 reações 0 responsáveis Ver no GitHub
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

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.