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

Use siane version to avoid potential issues #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dieghernan
Copy link
Member

Hola @hmeleiro:

He visto que en la viñeta de infoelectoral usas mapSpain para extraer los polígonos de los municipios a una fecha dada.

El riesgo es veo es que al usar year = "2016" se fuerza una descarga a través de giscoR. Esta descarga es bastante pesada (>100Mb) porque tiene todos los municipios de la UE (mapSpain luego filtra solo los de España) y además en el pasado ha habido problemas con el acceso a GISCO por cambios en el servidor de la API (rOpenGov/giscoR#74, rOpenGov/giscoR#69 y varios más).

Al incluirla en la viñeta, que CRAN ejecuta regularmente, veo cierto riesgo de que en un momento dado la viñeta dé problemas y tengas algún email de aviso desde CRAN por ERROR.

Este PR cambia esp_get_munic() por esp_get_munic_siane(), que descarga la información equivalente pero proporcionada por el IGN solo para España, y es mucho más ligera (5Mb). Los resultados son los mismos y esta función jamás me ha dado problemas.

Saludos.

@dieghernan dieghernan changed the title Use siane version to avoid issues Use siane version to avoid potential issues Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant