-
Notifications
You must be signed in to change notification settings - Fork 17
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
Added cultural 10m dataset #131
base: main
Are you sure you want to change the base?
Conversation
@Nageshbansal which tables are failing? |
ne_10m_admin_1_states_provinces |
I have created an issue nvkelso/natural-earth-vector#674. Let's see if the authors of the data can help. |
This reverts commit 3227759.
@henrykironde, Scripts contains updated paths now, only those falling 6 tables are remaining, can you tell me how can we proceed further now? |
We shall have to leave this PR open with hopes that the distributors of the data can change the dataset soon. |
Also when creating these scripts, do not change the structure / organization /arrangement of the keys in the file. |
@henrykironde can you please just review this PR once , and if dataset get updated then we can merge this PR. |
It looks like you may have over written the commits. Let's work on this together to get the files organized. This happens when you push always to main. We should be using other branches not main. Each issue should have a separate branch. |
Yeah sure, should i create a new PR for this on a separate branch. |
added the cultural 1:10m dataset provided by : https://www.naturalearthdata.com/downloads/10m-cultural-vectors/