pvlib codebase not compliant with flake8
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
Piste de recherche
The issue says CI only lints the lines changed in each PR, so the rest of the codebase has never been checked. Run flake8 over the whole repository and collect the reported violations, then review the existing exceptions the issue mentions. Done means flake8 runs clean on the full tree. The author planned to split this into two PRs, so check with maintainers which files are already covered before starting.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the bug
As mentioned by @echedey-ls in #2637, since the GitHub CI only covers the diffs within a PR, the codebase remains unchecked.
Expected behavior
Be flake8-compliant, although some nuances may raise exceptions (or existing exceptions may also be revised).
Versions:
pvlib.__version__: 0.13.1
Additional context
Will address this in two separate PRs to make it easier to review (one today, another some other time).
- 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 Une pull request liée à cette issue est ouverte ou déjà fusionnée. 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 15 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 252 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
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
SACGF/variantgrid#2341 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
comp/cli P3 type/security
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
NousResearch/hermes-agent#141420 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
CKV_GCP_18 (GKEPublicControlPlane) passes when master_authorized_networks_config is absent, control plane is internet-reachablePeut-être pris @Varshith-Kali l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
bridgecrewio/checkov#7753 ·
Les mainteneurs répondent en général sous 1 jour