test_utils.py uninstalls numpy
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 18.8k
- Forks
- 2.8k
- Ø Merge
- 16 Std. 26 Min.
- Gemergte PRs (30 T.)
- 21
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Lies packages/python/plotly/plotly/tests/test_optional/test_utils/test_utils.py etwa in den Zeilen 453–465 und vergleiche die Stelle mit dem Test, der in PR #4622 verschoben wurde. Ermittle, warum der Test numpy deinstalliert, und überprüfe anschließend, dass der überarbeitete Ansatz den Rekursionsfehler verhindert und andere optionale Tests nicht beeinträchtigt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- numpy, python
- Bereich
- testing-qa
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100