syt123450 / giojs

🌏 A Declarative 3D Globe Data Visualization Library built with Three.js

Home Page:https://giojs.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Missing countries

AlexisAnzieu opened this issue · comments

Hi!

When i tried to connect it with youtube stats, I had some issues with countrycode. I notice some missing countries from your singleton object :

I only found it for now but I will post update if there are more of them.

Thanks for the lib anyway !

Thanks for reminding us about the missing countries. We will add them to our library based on your suggestion. However, in order to add new countries, we need to update not only the country object but also the MapIndex data, which may take time.

Using "ZZ" to represent UNKNOWN countries is an amazing idea, and it is really exciting to add this feature to Gio.js. We can create another issue to work on it right now. Can you give us a hand? @AlexisAnzieu

Can you help to check and add missing countries in countryData object @BoTime

Can you help to update the mapIndex data for missing country? @lq3297401

yes no problem @syt123450.
I've created #88 and will work on it as soon as "ZZ" is added to the object and Index !

Thanks so much for your help ~ Looking forward for this feature! @AlexisAnzieu

@AlexisAnzieu Hi, I have added the missing country code: RE, HK, GP for Gio.js and update the lib in "build" folder. As HK is an inalienable part of the Chinese territory, every time referring to country code HK, Gio.js will highlight the whole Chinese territory. Thanks for your bug report, hoping this kind of fix can meet your requirement~

@syt123450 Great ! What about "ZZ" code ? I think that it could be placed in the middle of an ocean 🌊

@AlexisAnzieu How about ignore the json object in the list whose "e" or "i" is "ZZ"? Just my imagination, if we place the ZZ's position in a specific ocean, it has a sense that a country exports something into ocean. (o.o)

Hmmm sure ! Is it an issue if your ZZ object didn't get any geopoint location in your CountryData.js ?

@AlexisAnzieu Just a draft idea, can we check the "ZZ" in GeometryDataProcessor.js, and ignore the object which contains "ZZ", then write a markdown documentation about the usage of country code, and "ZZ" feature? I create a WIP pr #96 , which only has some basic pseudocode for this idea. If this idea is feasible, we can work together to complete it~