0xMiden/examples

Benchmarking CLI improvements

Aberta

#39 aberto em 21 de dez. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (1 responsável)TypeScript (25 forks)github user discovery
good first issue

Métricas do repositório

Stars
 (21 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

A few small things that would be great to improve in benchmarking CLI:

  1. We should use structop instead of clap as thats what we use for all other CLIs (e.g., here).
  2. We should keep CLI parameters as consistent as possible across different CLIs (e.g., for how we specify security, number of outputs etc.). See example here.
  3. Let's change the executable file name to something like miden-bench (miden-benchmarking-cli is way too long).
  4. miden::prove() also includes execution time, I believe. So, to make things more clear, we should probably label it as "Execution + proving time" or something like that.
  5. Would also be great to print out proof size.

Guia do colaborador

Direção de pesquisa
Examine a implementação atual da CLI de benchmark no repositório de exemplos. Identifique onde o clap é usado e substitua por structop, seguindo o padrão do main.rs do miden vm. Revise outras CLIs (por exemplo, prove.rs) para nomeação consistente de parâmetros. Renomeie o executável para miden bench. Atualize o rótulo de saída para incluir 'Execution + proving time'. Adicione a impressão do tamanho da prova.
Pilha de tecnologia
rust
Domain
clitooling
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75