IMF lower limit, variables defined during setup
@bd-j is already working on this.
Since Feb 28, 2021.
Assessment
This issue has not been assessed yet.
Description
The FSPS setup module reads imf_lower_bound from the imf_lower_limit variable at instantiation and uses the former variable when computing the weight of the lowest mass isochrone point. However, only the latter variable is accessible through python-fsps. In practice this means that changing imf_lower_limit to any value below the mass of the lowest-mass isochrone point leads to incorrect calculation of its weight.
The best fix for this is probably in fsps itself, but it's probably worth checking sps_setup.f90 for any similar cases.
- Dominant language
- Python
- Stars
- 76
- Forks
- 45
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from dfm/python-fsps
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
dfm/python-fsps#121 ·
-
Performance question Open
Difficulty 4/5 3-5 days Newbie friendliness 25/100
dfm/python-fsps#350 · 3 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
dfm/python-fsps#229 · 8 comments · 1 reaction ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
dfm/python-fsps#217 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 28/100
dfm/python-fsps#199 · 4 comments ·
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100