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.

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

Places: Ujungpandang, Indonesia, is now Makassar #398

Closed
jwva opened this issue Aug 16, 2020 · 5 comments
Closed

Places: Ujungpandang, Indonesia, is now Makassar #398

jwva opened this issue Aug 16, 2020 · 5 comments
Assignees

Comments

@jwva
Copy link

jwva commented Aug 16, 2020

In the Places table, the populated place of Ujungpandang (Sulawesi, Indonesia) is officially called Makassar since 1999. In addition, it is not only a "Populated place", it is the "Admin-1 capital" of South Sulawesi.
https://en.wikipedia.org/wiki/Makassar

@nvkelso nvkelso added this to the v5.0.0 milestone Sep 11, 2020
@nvkelso nvkelso modified the milestones: v5.0.0, v5.1.0 Feb 2, 2021
@elliotap elliotap self-assigned this Mar 4, 2021
@elliotap
Copy link
Collaborator

elliotap commented Mar 5, 2021

Verified - wikipedia indicates name was changed to Makassar in 1999
Indonesian name is Kota Makassar (https://id.wikipedia.org/wiki/Kota_Makassar)

Compare to Google:
Screen Shot 2021-03-04 at 3 59 13 PM

Apple:
Screen Shot 2021-03-04 at 4 00 45 PM

OpenStreetMap:
Screen Shot 2021-03-04 at 4 01 16 PM

Proposed edits:
featurecla from Populated place to Admin-1 capital
name from Ujungpandang to Makassar
namealt from Ujung Pandang to Kota Makassar
nameascii from Ujungpandang to Makassar
meganame from Ujung Pandang to Makassar
ls_name from Ujungpandang to Makassar

edits in 10m & 50m populated places

@mizmay
Copy link
Collaborator

mizmay commented Mar 7, 2021

Happy to pick this one up too @elliotap!

@nvkelso
Copy link
Owner

nvkelso commented Mar 9, 2021

Only the 10m file needs to be updated as follows (with my modifications):

  • featurecla to Admin-1 capital (from Populated place)
  • name to Makassar (from Ujungpandang)
  • namealt to Kota Makassar|Ujung Pandang
  • nameascii to Makassar (from Ujungpandang)

Please don't change

  • meganame
  • ls_name

mizmay added a commit to mizmay/natural-earth-vector that referenced this issue Apr 4, 2021
@mizmay
Copy link
Collaborator

mizmay commented Apr 4, 2021

values ind
3 SCALERANK
110 NATSCALE
1 LABELRANK
Admin-1 capital FEATURECLA
Makassar NAME
NA NAMEPAR
Kota Makassar|Ujung Pandang NAMEALT
Makassar NAMEASCII
0 ADM0CAP
NA CAPIN
0 WORLDCITY
1 MEGACITY
Indonesia SOV0NAME
IDN SOV_A3
Indonesia ADM0NAME
IDN ADM0_A3
Sulawesi Selatan ADM1NAME
ID ISO_A2
NA NOTE
-5.138013 LATITUDE
119.4300817 LONGITUDE
1262000 POP_MAX
1262000 POP_MIN
2085674 POP_OTHER
12 RANK_MAX
12 RANK_MIN
Ujung Pandang MEGANAME
Ujungpandang LS_NAME
2133664 MAX_POP10
2137598 MAX_POP20
2134816 MAX_POP50
2136185 MAX_POP300
0 MAX_POP310
100 MAX_NATSCA
1604 MIN_AREAKM
1686 MAX_AREAKM
619 MIN_AREAMI
651 MAX_AREAMI
1282 MIN_PERKM
1312 MAX_PERKM
797 MIN_PERMI
815 MAX_PERMI
119.358333 MIN_BBXMIN
119.358333 MAX_BBXMIN
119.908333 MIN_BBXMAX
119.908333 MAX_BBXMAX
-5.7 MIN_BBYMIN
-5.7 MAX_BBYMIN
-4.933333 MIN_BBYMAX
-4.933333 MAX_BBYMAX
119.595958 MEAN_BBXC
-5.378118 MEAN_BBYC
Asia/Makassar TIMEZONE
288 UN_FID
223 POP1950
283 POP1955
361 POP1960
399 POP1965
418 POP1970
502 POP1975
629 POP1980
719 POP1985
816 POP1990
926 POP1995
1051 POP2000
1194 POP2005
1262 POP2010
1374 POP2015
1559 POP2020
1713 POP2025
1837 POP2050
5 MIN_ZOOM
Q14634 WIKIDATAID
102018665 WOF_ID
0 CAPALT
Makassar NAME_EN
Makassar NAME_DE
Macasar NAME_ES
Makassar NAME_FR
Macáçar NAME_PT
Макасар NAME_RU
望加錫 NAME_ZH
NA LABEL
NA NAME_AR
NA NAME_BN
NA NAME_EL
NA NAME_HI
NA NAME_HU
NA NAME_ID
NA NAME_IT
NA NAME_JA
NA NAME_KO
NA NAME_NL
NA NAME_PL
NA NAME_SV
NA NAME_TR
NA NAME_VI
1159150843 NE_ID

mizmay added a commit to mizmay/natural-earth-vector that referenced this issue Apr 5, 2021
mizmay added a commit to mizmay/natural-earth-vector that referenced this issue Apr 5, 2021
mizmay added a commit to mizmay/natural-earth-vector that referenced this issue Apr 17, 2021
@nvkelso
Copy link
Owner

nvkelso commented Apr 22, 2021

Fixed via #508 and 54a6619 to reapply #5.

@nvkelso nvkelso closed this as completed Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants