POSYDON-code/POSYDON

Clean out duplicated functions and variables

Fermée

#442 ouverte le 21 nov. 2024

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Python (35 forks)auto 404
enhancementgood first issuev2 feature

Métriques du dépôt

Stars
 (42 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Identifiez toutes les occurrences des fonctions et variables dupliquées listées, puis consolidez les en une seule definition en important depuis la source canonique.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60