Multiple StellarPopulation instances
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 76
- Forks
- 45
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Because parameter state and dirtiness is specific to each instance, but the computed model is held by the module level driver object, having multiple StellarPopulation instances can cause a model to not be recomputed even though it should be (example code below) This might also cause problems with shared memory multiprocessing.
Some documentation advising only a single sps instance per program might be a good idea.
from numpy.testing import assert_allclose
import fsps
sps = fsps.StellarPopulation(zcontinuous=1, vactoair_flag=False)
pop = fsps.StellarPopulation(zcontinuous=1, vactoair_flag=False)
w1, s1 = sps.get_spectrum(tage=10.0)
w, s = pop.get_spectrum(tage=1.0)
w2, s2 = sps.get_spectrum(tage=10.0)
assert_allclose(s, s2) # this should not pass, but does
assert_allclose(s1, s2) # this should not fail, but does
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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.
Piste de recherche
Commencez par StellarPopulation.get_spectrum et par l’état du driver au niveau du module décrit dans l’issue. Reproduisez la séquence fournie avec deux instances, puis suivez la manière dont la modification des paramètres et le recalcul du modèle sont partagés. Le travail est terminé lorsque chaque instance renvoie des résultats cohérents avec ses propres paramètres, y compris les deux assertions de l’exemple.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- fortran, python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100