Absolutely not thread-safe usage of cwd · nteract/papermill

(5 commenti) (0 reazioni) (0 assegnatari)Python (402 fork)batch import

enhancementhelp wantednew-contributor-friendly

Metriche repository

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

Descrizione

This part of code is absolutely not thread-safe: https://github.com/nteract/papermill/blob/68580f79fd7cc037dd3908343e3a070c3140395b/papermill/execute.py#L91-L105

Why don't you use cwd parameter in nbclient.client.execute or at least copy this function? https://github.com/jupyter/nbclient/blob/8d5fc8caa0b27b308760e5a350369720c03efc8b/nbclient/client.py#L730-L750

Guida contributor

Direzione di ricerca: Sostituisci os.chdir non sicuro con il parametro cwd in nbclient.client.execute. Controlla le righe 91-105 di papermill/execute.py e modifica per usare l'argomento cwd come mostrato nella documentazione di nbclient.
Tech stack: python
Dominio: backend
Tipo issue: Bug
Difficoltà: 2
Tempo stimato: 1-3 ore
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: Nessuno
Adatta ai principianti: 75