nteract/papermill

Injecting `pm` parameters breaks.

Aberta

#640 aberto em 8 de nov. de 2021

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)Python (402 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (5.381 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

🐛 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

Guia do colaborador

Direção de pesquisa
Investigue como o papermill lida com a injeção de parâmetros em caminhos de saída. O problema indica que, embora nomes de parâmetros simples funcionem em strings de formato (ex.: `{PARAM}`), o acesso no estilo de dicionário como `{pm['run uuid']}` falha. Revise o código relevante na lógica de substituição de parâmetros do papermill, provavelmente nos módulos de execução ou processamento de notebook. O PR vinculado #343 fornece contexto adicional sobre as tentativas de correção. Concentre se no mecanismo de formatação de strings e em como os parâmetros `pm` são passados para a função de formatação.
Pilha de tecnologia
python
Domain
cli
Tipo Issue
Bug
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
GitPython
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75