nteract/papermill

\r characters replaced by \r\n

Open

#722 ouverte le 27 avr. 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Python (402 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (5 381 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

🐛 Bug

Consider the following cell: image When running normally (i.e., through interactive python), each time a new line is printed, the \r put the cursor back to the start of the line which is replaced by the next print, and the final output of the cell should be only the last print: Hello World! 20

However if I use papermill to run the notebook with I get the following output: image

This is also affecting tdqm progress bar which output becomes an unreadable mess: image

Is someone else encountering the issue ? I'm using it inside a Docker installation.

Guide contributeur

Direction de recherche
Étudiez comment papermill capture la sortie du notebook, en particulier la gestion de stdout et stderr dans le pipeline d'exécution. Examinez le code dans `papermill/execute.py` ou des modules similaires pour tout traitement des caractères de retour chariot (`\r`). Vérifiez s'il y a un problème de conversion de fin de ligne qui remplace `\r` par `\r\n`. Considérez l'effet de l'environnement Docker sur le comportement des fins de ligne. Passez en revue les commentaires existants ou les issues connexes pour un contexte supplémentaire.
Stack technique
python
Domaine
backendcli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonJupyter Notebooks
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70