Simplify or remove `assert_*_equal` wrappers around `pandas.testing` functions
Les mainteneurs répondent en général sous 1 jour
@dgowdaan-cmyk y travaille déjà.
Depuis le 8/7/2026.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 62/100
Piste de recherche
Les wrappers de assert_frame_equal, assert_series_equal et assert_index_equal se trouvent dans conftest.py, où ils vérifient la version de pandas par rapport à 1.1.0 et ajustent les kwargs. Commence par lire ces trois wrappers et les tests qui les appellent. C'est terminé lorsque la vérification de version et la branche de kwargs sont supprimées, que les wrappers passent toujours la suite de tests existante et que la version minimale de pandas est confirmée supérieure à 1.1.0. Remplacer les wrappers par des appels directs à pandas est une question plus vaste et distincte, qu'il vaut mieux laisser à la décision d'un mainteneur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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).
- Langage dominant
- Python
- Étoiles
- 1.7k
- Forks
- 1.2k
- Merge moyen
- 3 j 18 h
- PR mergées (30 j)
- 13
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de pvlib/pvlib-python
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
pvlib/pvlib-python#2903 · 5 commentaires · 2 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
use offset parameter instead of calculating bday in cdromPeut-être pris @chenyi6758-source l’a pris il y a 1 jour. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
pvlib/pvlib-python#2902 ·
Les mainteneurs répondent en général sous 1 jour
-
Add an example or documentation on using 'bhi' from ERA5Peut-être à nouveau libre @anshurajbisoyi98-ctrl l’a pris il y a 16 jours, et aucune pull request n’est ouverte. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
pvlib/pvlib-python#2880 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Peut-être à nouveau libre @Rahi-padwal l’a pris il y a 254 jours, et aucune pull request n’est ouverte. Ouverteapi easy
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
pvlib/pvlib-python#2653 · 6 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
question
Difficulté 1/5 1-3 heures Accessibilité débutants 72/100
pvlib/pvlib-python#2637 · 10 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de pvlib/pvlib-python
Issues similaires
-
bug documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
awslabs/visual-asset-management-system#514 ·
Les mainteneurs répondent en général sous 1 jour
-
backend bug spec:REQ-053
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
nolte/kamerplanter#2547 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
EleutherAI/lm-evaluation-harness#4425 ·
Les mainteneurs répondent en général sous 1 jour
-
Gemini: respect PLANNING_DISABLED=1 in lifecycle context hooksPeut-être pris @AmariAugustin l’a pris aujourd’hui. Ouvertebug good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 87/100
OthmanAdi/planning-with-files#329 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
wr.dynamodb.read_items returns [{}] for a single partition value that does not existPeut-être pris @auraseb l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
aws/aws-sdk-pandas#3520 ·
Les mainteneurs répondent en général sous 2 jours