owid / etl

A compute graph for loading and transforming OWID's data
https://docs.owid.io/projects/etl
MIT License
86 stars 23 forks source link

🐝 Update regions.yml to shorten Turks and Caicos Islands #3504

Closed spoonerf closed 2 weeks ago

spoonerf commented 2 weeks ago

Hey @pabloarosado an easy PR, but I'm not sure if this will trigger a large rebuild of the ETL, or is that something of the past?

owidbot commented 2 weeks ago
Quick links (staging server): Site Admin Wizard Docs

Login: ssh owid@staging-site-shorten-turks-and-caicos

chart-diff: ✅ No charts for review.
data-diff: ❌ Found differences ```diff = Dataset garden/regions/2023-01-01/regions = Table regions ~ Column short_name (changed data) ~ Changed values: 2 / 312 (0.64%) code short_name - short_name + BIH Bosnia and Herzegovina Bosnia and Herz. TCA Turks and Caicos Islands Turks and Caicos Legend: +New ~Modified -Removed =Identical Details Hint: Run this locally with etl diff REMOTE data/ --include yourdataset --verbose --snippet ``` Automatically updated datasets matching _weekly_wildfires|excess_mortality|covid|fluid|flunet|country_profile|garden/ihme_gbd/2019/gbd_risk_ are not included

Edited: 2024-11-06 16:14:39 UTC Execution time: 982.93 seconds

spoonerf commented 2 weeks ago

Ah good point, it is mostly in order to make this chart less squashed. I think just removing 'Islands' will help a lot. Although now looking at it, 'Bosnia and Herzegovina' is also quite long, just two characters less.

I'll ask on #data-and-research about appropriate short names e.g. would it be appropriate to shorten the latter to just 'Bosnia'.