"upload" only works with "compiled" benchmarks
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 1k
- Fork
- 203
- Merge medio
- 1h 20m
- PR unite (30g)
- 2
Descrizione
My workflow for PyPy is
- download latest version compiled on buildbot,
- run benchmarks,
- upload data (using command line option
upload, which goes throughcompile.py, but should it?.
The cmd_upload function in compile.py assumes BenchmarkRevision.update_metadata from compile.py has been run to add commit information (commit_id, commit_branch, commit_date) and possibly patch_file to the json benchmark info. But because I am not running compile this info is lacking. Possible solutions:
- add a
download_prebuiltoption tocompile.pyand thePythonclass to support this workflow - refactor the
update_metadatamethod fromBenchmarkRevisionto become part of every benchmark run
Any thoughts? Should there be an upload.py separate from compile.py?
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia leggendo compile.py, in particolare cmd_upload, BenchmarkRevision.update_metadata e la classe Python. Traccia le differenze tra i benchmark Buildbot scaricati e i benchmark compilati e il modo in cui i metadati vengono aggiunti prima dell’upload. Il lavoro è completato quando il workflow PyPy descritto può caricare i dati dei benchmark senza richiedere un passaggio di compilazione locale e la separazione o il refactoring scelto è coperto dai test pertinenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- performance, tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 30/100