Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update demographics.py #1

Closed
wants to merge 2 commits into from
Closed

Update demographics.py #1

wants to merge 2 commits into from

Conversation

sarvonz
Copy link

@sarvonz sarvonz commented Nov 23, 2022

@rickecon This is a PR regarding adding code that retrieves neonatal mortality rate from WB WDI

@sarvonz
Update demographics.py
f373110 
add code that retrieves neonatal mortality rate from WB WDI
@sarvonz
black formatted demographics.py
46dd688
@rickecon rickecon mentioned this pull request Dec 5, 2022
Merged
@rickecon
Copy link
Owner

rickecon commented Dec 5, 2022

@sarvonz @SeaCelo. In contrast to the approach of this PR, I decided to make a new function get_wb_infmort_rate() in demographics.py in my branch for PR #14 and to put the call to the infant mortality rate in the get_mort() function. So we can close this PR.

@rickecon rickecon closed this Dec 5, 2022
rickecon pushed a commit that referenced this pull request Jan 13, 2023
@SeaCelo
Merge pull request #1 from rickecon/SeaCelo-yield-wedge
2b91820 
Updated three initial guesses in ogzaf_default_parameters.json
rickecon pushed a commit that referenced this pull request Jun 18, 2024
@jdebacker
Merge pull request #1 from SeaCelo/jdebacker-updates_ogcore
bf76bd6 
Update macro_params.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sarvonz @rickecon