POSYDON-code/POSYDON

Clean out duplicated functions and variables

Aperta

#442 aperta il 21 nov 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (35 fork)auto 404
enhancementgood first issuev2 feature

Metriche repository

Star
 (42 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We have different functions and variables which have duplicates. We should clean that up to always use one version. Here some examples

  • posydon.utils.gridutils.kepler3_a vs. posydon.utils.common_functions.orbital_separation_from_period
  • posydon.utils.gridutils.T_merger_P vs. posydon.utils.common_functions.inspiral_timescale_from_orbital_period
  • posydon.utils.gridutils.T_merger_a vs. posydon.utils.common_functions.inspiral_timescale_from_separation
  • variables are often hard coded in functions instead of imported from where they are defined, this happens often in step_* vs. flow_chart

Guida contributor

Direzione di ricerca
Identifica tutte le occorrenze delle funzioni e variabili duplicate elencate, quindi consolidale in un'unica definizione importando dalla fonte canonica.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60