Consider switching to `src` layout
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
Piste de recherche
The package layout change from #2277 is the starting point, so compare the repository tree against the src layout pattern and check the packaging config in pyproject.toml or setup.cfg. Moving the package under src/ touches imports, test discovery and the documentation build, so run the test suite afterwards. Done means the package installs from src/, the tests pass, and the maintainers agree on the final layout.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
To my mind, src is a separate discussion, and doesn't need to be done in https://github.com/pvlib/pvlib-python/pull/2277, if at all.
I brought it up because it seems to me that the majority of projects that use
/testsalso use/src. Indeed most of the linked packages on the PyOpenSci reference use/pkg/testsrather than/pkg+/tests. Alsopytestdocs include examples for either/src+/testsOR/pkg/testsbut not/pkg+/tests. Maybe it's ok but I like to follow the more popular patterns unless I know why I am deviating from them.
Originally posted by @wholmgren in #2271
#2277 moved the package from a flat layout to a "halfway-src" layout. We should consider adopting the src layout entirely, as the python community seems broadly agreed that the src layout is beneficial.
- 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 253 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
-
good first issue tools
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Commits pushed after #1095 merged did not land (ahmi/unified-nova-mcp-review-e095 @ 80fa23a)Ouvertebug deferred domain:ci P1
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
get_page_thumbnail modifies cached page images, affecting later thumbnails and picture exportsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
docling-project/docling-mcp#147 ·
-
issue: A pipe model's Valves button ignores the Allow Chat Valves permissionPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug confirmed issue
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
open-webui/open-webui#32771 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Scheduled Security Scan is failing on mainPeut-être pris @pkupt l’a pris aujourd’hui. Ouvertesecurity-scan-failure
Difficulté 1/5 1-3 heures Accessibilité débutants 74/100
semantica-agi/semantica#2045 ·
Les mainteneurs répondent en général sous 2 jours