nteract/papermill

Injecting `pm` parameters breaks.

Open

#640 aperta il 8 nov 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (402 fork)batch import
bughelp wanted

Metriche repository

Star
 (5381 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

🐛 Bug

This works nicely for injecting params into the output_path (output name).

papermill template.ipynb analysis_{PARAM}.ipynb -p PARAM test_stuff

however, using pm['run_uuid'] breaks as does pm["run_uuid"] and pm[run_uuid]. It seems that python doesn't allow access to the pm object values in the "".format() syntax.

papermill template.ipynb analysis_{pm['run_uuid']}.ipynb

using pm alone works and prints the entire pm dict in the notebook output name, but it very messy!

See here for more info: https://github.com/nteract/papermill/pull/343

Guida contributor

Direzione di ricerca
Indaga su come papermill gestisca l'iniezione di parametri nei percorsi di output. Il problema indica che mentre i nomi di parametri semplici funzionano nelle stringhe di formato (es. `{PARAM}`), l'accesso in stile dizionario come `{pm['run uuid']}` fallisce. Rivedi il codice pertinente nella logica di sostituzione dei parametri di papermill, probabilmente nei moduli di esecuzione o elaborazione notebook. La PR collegata #343 fornisce ulteriore contesto sui tentativi di correzione. Concentrati sul meccanismo di formattazione delle stringhe e su come i parametri `pm` vengono passati alla funzione di formato.
Tech stack
python
Dominio
cli
Tipo issue
Bug
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
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75