plotly / plotly/plotly.py

setting up a development environment is difficult

Aperta
#4,380 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug infrastructure P2
Lingua principale
Python
Stelle
18.8k
Fork
2.8k
Merge medio
16h 26m
PR unite (30g)
21

Descrizione

I recently spun up a dev environment for contributing a PR to plotly.py and ran into some issues that I think can be addressed fairly readily.

  1. pytest is pinned to 3.5.1 in packages/python/plotly/optional-requirements.txt and this version simply didn't run, failing with an internal TypeError. I upgraded it to 7.4.2 and it worked. Is there a reason its pinned to such an old version? can this be unpinned?
  2. contributing.md does not currently mention the dependancy on Plotly Orca. I was rather surprised to come back to all my tests failing, and somewhat hilariously the orca Ubuntu screen reader reading a large chunk of Pytest's terminal output to me. Just adding an external dependency in the testing section would do the trick I reckon.
  3. There were a number of missing packages required by optional tests that aren't in packages/python/plotly/optional-requirements.txt so those tests failed. The following packages need to be added: polars vaex scikit-image statsmodels pyarrow

btw for 1,the error thrown by pytest 3.5.1 is:

  File "/home/ned/.pyenv/versions/3.11.6/envs/book/lib/python3.11/site-packages/_pytest/assertion/rewrite.py", line 332, in _rewrite_test
    co = compile(tree, fn.strpath, "exec", dont_inherit=True)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: required field "lineno" missing from alias

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia con packages/python/plotly/optional-requirements.txt e contributing.md, quindi riproduci l’errore di pytest 3.5.1 nell’ambiente Python 3.11 segnalato. Controlla le dipendenze dei test opzionali e la documentazione sui test. Il lavoro è completato quando la configurazione documentata esegue i test senza l’errore di pytest segnalato o errori dovuti a pacchetti mancanti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
build-system, documentation, testing
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.