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

Topónimos colombianos #120

Merged
merged 5 commits into from Jul 11, 2017

Conversation

Projects
None yet
3 participants
@cosmoscalibur
Collaborator

cosmoscalibur commented Jun 2, 2017

Buen día.
He realizado la lista de topónimos colombianos acorde a la última actualización del DIVIPOLA (30 de septiembre de 2015) siguiendo los parámetros estipulados en la wiki (#81).
Igualmente dejo un registro metodológico para el mantenimiento de estos archivos y una rutina bash de apoyo para la actualización (que podría verse afectado por un posible cambio de estructura en una futura actualización del DIVIPOLA).
Me queda la inquietud respecto a incluir o no la parte de los topónimo compuesto que corresponden a números romanos (presento 6 lemas con dichas características).

@sbosio

sbosio approved these changes Jul 8, 2017

Todos los cambios me parecen correctos, solo haría un ligero cambio respecto de los números romanos que mencionabas. De incluirlos como combinaciones de letras latinas las pondría todas en mayúsculas, creo que sería lo correcto. Otra opción es incluirlos como caracteres Unicode ahora que el diccionario está escrito en UTF-8. Por ejemplo: http://www.fileformat.info/info/unicode/char/2162/index.htm

@cosmoscalibur

This comment has been minimized.

Show comment
Hide comment
@cosmoscalibur

cosmoscalibur Jul 10, 2017

Collaborator

@sbosio , en ese caso al ser caracteres de bloque numérico sería lo mismo que pretender hacer revisión de números. Si te parece, considero no incluirlos (de entrada por defecto la revisión de números no se habilita en muchas aplicaciones así hunspell los trate como cualquier carácter).
Si estas de acuerdo, sería para unir el ramal. Muchas gracias.

Collaborator

cosmoscalibur commented Jul 10, 2017

@sbosio , en ese caso al ser caracteres de bloque numérico sería lo mismo que pretender hacer revisión de números. Si te parece, considero no incluirlos (de entrada por defecto la revisión de números no se habilita en muchas aplicaciones así hunspell los trate como cualquier carácter).
Si estas de acuerdo, sería para unir el ramal. Muchas gracias.

@sbosio

This comment has been minimized.

Show comment
Hide comment
@sbosio

sbosio Jul 10, 2017

Owner

@cosmoscalibur: Sí, es la tercera opción, directamente excluirlos. Usar los caracteres especiales Unicode es complicado porque no hay forma de ingresarlos con el teclado. Lo más común es ingresarlos con letras latinas en mayúsculas, como en "siglo XIX", y en ese caso lo marca como error, aunque no creo que confunda a nadie. Otra opción es a futuro armar un listado con todos los números romanos hasta cierto valor.
Avísame cuando los hayas eliminado para integrar los cambios.
¡Saludos!

Owner

sbosio commented Jul 10, 2017

@cosmoscalibur: Sí, es la tercera opción, directamente excluirlos. Usar los caracteres especiales Unicode es complicado porque no hay forma de ingresarlos con el teclado. Lo más común es ingresarlos con letras latinas en mayúsculas, como en "siglo XIX", y en ese caso lo marca como error, aunque no creo que confunda a nadie. Otra opción es a futuro armar un listado con todos los números romanos hasta cierto valor.
Avísame cuando los hayas eliminado para integrar los cambios.
¡Saludos!

@cosmoscalibur

This comment has been minimized.

Show comment
Hide comment
@cosmoscalibur

cosmoscalibur Jul 11, 2017

Collaborator

Hola @sbosio, ya he retirado los lemas de números romanos. Cualquier cosa me comentas, de lo contrario ya esta listo para la unión. Buen día.

Collaborator

cosmoscalibur commented Jul 11, 2017

Hola @sbosio, ya he retirado los lemas de números romanos. Cualquier cosa me comentas, de lo contrario ya esta listo para la unión. Buen día.

@sbosio sbosio merged commit 18b776c into sbosio:master Jul 11, 2017

@sbosio

This comment has been minimized.

Show comment
Hide comment
@sbosio

sbosio Jul 11, 2017

Owner

@cosmoscalibur: acabo de integrar los cambios. Felicitaciones por el gran aporte que has hecho. Aún no hemos fijado fecha para publicar la versión 2.3, pero estos cambíos estarán integrados en esa versión.

Owner

sbosio commented Jul 11, 2017

@cosmoscalibur: acabo de integrar los cambios. Felicitaciones por el gran aporte que has hecho. Aún no hemos fijado fecha para publicar la versión 2.3, pero estos cambíos estarán integrados en esa versión.

@edittler edittler added this to the Versión 2.3 milestone Jul 29, 2017

@cosmoscalibur cosmoscalibur referenced this pull request Jan 3, 2018

Closed

Publicación de la versión 2.3 #153

4 of 4 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment