Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Notebook tooling: clear_outputs.py stops at the first dirty cell, wasm_check does not test the built wheel

Ouverte
  • Quelque chose ne va pas ?
#157 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
42/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
jupyter, python, wasm
Domaine
testing, tooling

Piste de recherche

Start with docs/notebooks/clear_outputs.py and its process_notebook function, then inspect docs/lite/wasm_check/check_pyodide_install.mjs alongside _mvtb_nb_bootstrap.py. Check that every dirty cell is cleared and that the wheel is written where the bootstrap searches; the issue says both fixes are already present in the RTB copy, so use its notebook hygiene PR as a reference. Done when both tools operate on all dirty cells and the wheel built for the check.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Likely smells in the notebook tooling, found while copying it to RTB. Worth checking out and fixing.

  1. docs/notebooks/clear_outputs.py, process_notebook: changed = any(clear_cell(cell) for cell in nb.get("cells", [])) is a generator, and any() stops at the first cell that returns True. Only the first dirty cell of each notebook is cleared, so a second run clears the next one, and so on. Use a list: any([clear_cell(c) for c in cells]).

  2. docs/lite/wasm_check/check_pyodide_install.mjs writes the wheel to /<wheel> in Pyodide's filesystem, but _mvtb_nb_bootstrap.py looks for it with Path("/pypi").glob(...). It finds nothing, so it falls back to installing machinevision-toolbox-python from PyPI and the check tests the last release, not the wheel just built. Also write the wheel to /pypi/<wheel> (FS.mkdirTree("/pypi")).

Both are fixed in the RTB copy, see the notebook hygiene PR in robotics-toolbox-python.

Langage dominant
Python
Étoiles
224
Forks
35
Merge moyen
1 j 20 h
PR mergées (30 j)
34

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de petercorke/machinevision-toolbox-python

Toutes les issues de petercorke/machinevision-toolbox-python

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.