Simplify or remove `assert_*_equal` wrappers around `pandas.testing` functions
I maintainer di solito rispondono entro 1 giorno
@dgowdaan-cmyk ci sta già lavorando.
Dal 8/7/2026.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 62/100
Direzione di ricerca
I wrapper per assert_frame_equal, assert_series_equal e assert_index_equal si trovano in conftest.py, dove verificano la versione di pandas rispetto a 1.1.0 e adattano i kwargs. Inizia leggendo questi tre wrapper e i test che li richiamano. È completato quando il controllo della versione e la diramazione sui kwargs sono rimossi, i wrapper continuano a superare la suite di test esistente e la versione minima di pandas è confermata superiore a 1.1.0. Sostituire i wrapper con chiamate dirette a pandas è una questione più ampia e separata, meglio lasciarla a una decisione di un maintainer.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
We have our own wrappers in conftest.py for the three pandas assert_X_equal functions. We originally added those because the default tolerances of those functions changed in pandas 1.1.0 (see #1018). Our wrappers check the version of pandas and do some simple kwarg manipulation based on whether the installed pandas is greater or lower than 1.1.0.
Now that our minimum pandas is greater than 1.1.0, the version checking is no longer needed and we can simplify those functions.
It is also worth looking into whether we can/should get rid of those wrappers entirely and just use the pandas functions directly, but that change might require a lot of work (https://github.com/pvlib/pvlib-python/issues/1018#issuecomment-668631386).
- Lingua principale
- Python
- Stelle
- 1.7k
- Fork
- 1.2k
- Merge medio
- 3g 18h
- PR unite (30g)
- 13
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di pvlib/pvlib-python
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pvlib/pvlib-python#2903 · 5 commenti · 2 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
use offset parameter instead of calculating bday in cdromForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
pvlib/pvlib-python#2902 ·
I maintainer di solito rispondono entro 1 giorno
-
Add an example or documentation on using 'bhi' from ERA5Forse di nuovo libera @anshurajbisoyi98-ctrl l’ha presa 15 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
pvlib/pvlib-python#2880 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Forse di nuovo libera @Rahi-padwal l’ha presa 252 giorni fa e non c’è nessuna pull request aperta. Apertaapi easy
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
pvlib/pvlib-python#2653 · 6 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
question
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
pvlib/pvlib-python#2637 · 10 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di pvlib/pvlib-python
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
google-ai-edge/litert-torch#1346 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
geneontology/gocam-py#272 ·
-
pushed_authorization_request sends list authorization_details as a Python reprForse già presa @RaphaelFakhri l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
auth0/auth0-python#903 ·
I maintainer di solito rispondono entro 1 giorno