Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

library reorganization

Aperta
#46 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
15/100
Tipo di issue
Refactoring
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
python
Ambito
tooling

Direzione di ricerca

Inizia mappando gli import e gli utilizzi in calibration.py, basic.py, metacal.py, ellipticity.py, stats.py, io.py, util.py, survey.py, galaxy.py, catalog.py e nel codice relativo alla cosmologia e agli ammassi di galassie. Risolvi le questioni strutturali aperte con i maintainer, quindi definisci la disposizione dei moduli concordata e verifica che le funzionalità esistenti rimangano disponibili dopo la riorganizzazione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

clean-up

I went through the library. I will put a first pass at some changes I think we should make to the global structure in this issue.

  1. calibration.py should perform local and global metacal calibration
  2. let's move the metacal class out of basic.py into a separate metacal.py and get rid of basic.py completely
  3. add an ellipticity.py that switches between e1, e2, and e_t, e_x, as well as between different ellipticity conventions, this will help with handling different input catalogs
  4. stats.py will handle all of the stats functions in basic.py
  5. should the metacal class interact directly with catalogs? i feel like some of this should go into io.py
  6. util.py should be split into binning functions and vos functions
  7. survey.py has to do with the footprint, should we combine this with galaxy.py? at any rate, galaxy.py must be renamed. also, the conversion between fwhm and sigma belongs in stats
  8. it makes sense to group all cosmology things in one place, but i think there needs to be a further separation of the theory and data functionality.
  9. galaxy cluster functions needs to be together
  10. catalog.py again is a mix of stats and io, though it does perform a number of initial sanity checks. maybe we should rename it to cat_quick_check or something along those lines?

comments welcome- I'm sure I didn't consider every use case, so there are probably adaptations of the above that will make more sense.

Lingua principale
Python
Stelle
2
Fork
5
Merge medio
1g 3h
PR unite (30g)
40

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di CosmoStat/sp_validation

Tutte le issue di CosmoStat/sp_validation

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.