Skip to content

🎉 static viz: population over the long run#5602

Merged
veronikasamborska1994 merged 9 commits intomasterfrom
pop-growth-BCE
Feb 12, 2026
Merged

🎉 static viz: population over the long run#5602
veronikasamborska1994 merged 9 commits intomasterfrom
pop-growth-BCE

Conversation

@veronikasamborska1994
Copy link
Contributor

No description provided.

@owidbot
Copy link
Contributor

owidbot commented Feb 2, 2026

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-pop-growth-BCE

chart-diff: ✅ No charts for review.
data-diff: ✅ No differences found

Automatically updated datasets matching excess_mortality|covid|fluid|flunet|country_profile|garden/ihme_gbd/2019/gbd_risk are not included

Edited: 2026-02-02 15:07:16 UTC
Execution time: 9.87 seconds

Copy link
Member

@lucasrodes lucasrodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Veronika! Left some comments. Let me know if you need more feedback / clarifications.

Overall I think that the code is not up to ETL standards at the moment. I'd suggest dropping the hardcoded catalog paths, and possibly simplifying some bits here and there.

Again, let me know if you need any more reviews! Happy to help.

lucasrodes

This comment was marked as duplicate.

Copy link
Member

@lucasrodes lucasrodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Veronika! Left some comments. Let me know if you need more feedback / clarifications.

Overall I think that the code is not up to ETL standards at the moment. I'd suggest dropping the hardcoded catalog paths, and possibly simplifying some bits here and there.

Again, let me know if you need any more reviews! Happy to help.

@veronikasamborska1994 veronikasamborska1994 merged commit 9adcffd into master Feb 12, 2026
4 of 5 checks passed
@veronikasamborska1994 veronikasamborska1994 deleted the pop-growth-BCE branch February 12, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants