0xMiden/examples

Benchmarking CLI improvements

Aperta

#39 aperta il 21 dic 2022

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (1 assegnatario)TypeScript (25 fork)github user discovery
good first issue

Metriche repository

Star
 (21 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente della CLI di benchmarking nel repository degli esempi. Identifica dove viene utilizzato clap e sostituiscilo con structop, seguendo il pattern in main.rs di miden vm. Rivedi altre CLI (es. prove.rs) per una denominazione coerente dei parametri. Rinomina l'eseguibile in miden bench. Aggiorna l'etichetta di output per includere 'Execution + proving time'. Aggiungi la stampa della dimensione della prova.
Tech stack
rust
Dominio
clitooling
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75