-
Notifications
You must be signed in to change notification settings - Fork 673
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
Splitting out the list of providers in a standalone JSON file #457
Comments
Hi @SylvainCorlay nice idea, we just have to find a way to generate automagically this JSON file from the repo. Do you have any hint for that ? @jieter any thoughts on this ? |
Hi @brunob, we already have that magic in |
@martinfleis do you think we can use your script on our repo with a github ci or anything like that ? |
@brunob Yeah, we do that within this GHA https://github.com/geopandas/xyzservices/blob/main/.github/workflows/update_providers.yaml using this makefile |
Nice, let's hope that @jieter could look into it or propose an alternative method :) |
We could generate this JSON as a one-off, include the JSON in your repository, and then read the JSON from your JavaScript instead of having the object hard coded in the code base. This would make it a single source of truth. |
Yes, it would be nice to maintain the list in a machine readable format, and generate Other options would be JSONC, but that would require consumers of the file to use non-standard parsers too. Any suggestions? |
What about using YAML instead if we need comments? That feels as a bit more standard solution than JSONC. |
The big advantage of JSON is that you can parse it from either Python or JS without a third-party dependency. |
+1 for JSON too |
One option to keep comments in the JSON without using non-standard parsers would be to include an optional |
We would also like this at TileProviders.jl (which is used in Leaflet.jl but is generalised so everyone can use it) Currently we pull json from geopandas xyzservices because its easier to parse To minimise downstream workload I suggest that rather than YAML or a new JSON format we use the JSON format xyzservices has already defined. |
Currently, the available providers are listed in a JavaScript object that is defined inline in the source code of leaflet-providers.
It would be really useful to make that information available directly in a JSON file that could be used from other projects. For example, xyzservices extracts that information from leaflet-provider.js to make the same information available to Python. Xyzservices has some adoption in the scientific Python ecosystem (Geopandas, ipyleaflet, contextily, leafmap, bokeh, and others) - meaning that this project has in fact a lot of indirect adoption...
This separate JSON file could even be published independently of the leaflet-providers JS source, which would probably make it valuable for other users.
The text was updated successfully, but these errors were encountered: