Consider switching to `src` layout
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 25/100
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- 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 @chenyi6758-source l’ha presa oggi. 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 253 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
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
mohitmishra786/readprism#322 ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
API-key credentials accept a blank key: empty via the API, spaces-only via the builder's Add API key dialogForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
Significant-Gravitas/AutoGPT#15727 ·
I maintainer di solito rispondono entro 1 giorno