plotly / plotly/plotly.py

test_utils.py uninstalls numpy

Ouverte
#4,852 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

P2 testing
Langage dominant
Python
Étoiles
18.8k
Forks
2.8k
Merge moyen
16 h 26 min
PR mergées (30 j)
21

Description

In #4622, I encountered a very odd recursion error, and the solution was to move the test that was failing to the top of the file. This was because the test above it was uninstalling numpy. We should investigate why this test is even doing that and potentially change the approach to prevent similar issues.

Guide de contribution

Ouvrir le guide de contribution

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.

Piste de recherche

Lisez packages/python/plotly/plotly/tests/test_optional/test_utils/test_utils.py autour des lignes 453–465 et comparez-le avec le test déplacé dans PR #4622. Déterminez pourquoi le test désinstalle numpy, puis vérifiez que l’approche révisée évite l’erreur de récursion et ne perturbe pas les autres tests optionnels.

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

Évaluation

Stack technique
numpy, python
Domaine
testing-qa
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
38/100

Recevez les nouvelles issues par e-mail

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