Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Quitando palabras repetidas en cada diccionario #1699

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

Merged
merged 2 commits into from
Dec 11, 2021
Merged

Quitando palabras repetidas en cada diccionario #1699

merged 2 commits into from
Dec 11, 2021

Conversation

cmaureir
Copy link
Collaborator

@cmaureir cmaureir commented Dec 9, 2021

Este cambio solo hace modificaciones por archivo,
en caso que existan palabras repetidas.

No es una limpieza que considera todos los archivos,
pues la idea es detectar las palabras asociadas
a cada archivo que no fueron reconocidas por defecto.

Este cambio solo hace modificaciones por archivo,
en caso que existan palabras repetidas.

No es una limpieza que considera todos los archivos,
pues la idea es detectar las palabras asociadas
a cada archivo que no fueron reconocidas por defecto.
Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me parecen bien los cambios. Preguntas: 1) queremos forzar en el futuro que las entradas en los diccionarios estén ordenadas alfabéticamente? Y 2) si ordenamos alfabéticamente creo que sería mejor si combinamos mayúsculas y minúsculas en lugar de considerar todas las mayúsculas como menor que todas las minúsculas.

@cmaureir
Copy link
Collaborator Author

pues no lo sé...quizás (1) le produciría muchos problemas a las personas que comienzan, creo que con una limpieza de vez en cuando podría ser suficiente...mal que mal, cuando hacemos "el gran diccionario" mezclamos todo y eliminamos repetidas.

(2) Buen punto lo de dejar juntas mayúsculas y minúsculas, ¿lo aplico a estos archivos? o lo dejamos para otra vez?

@rtobar
Copy link
Collaborator

rtobar commented Dec 11, 2021

pues no lo sé...quizás (1) le produciría muchos problemas a las personas que comienzan, creo que con una limpieza de vez en cuando podría ser suficiente...mal que mal, cuando hacemos "el gran diccionario" mezclamos todo y eliminamos repetidas.

(2) Buen punto lo de dejar juntas mayúsculas y minúsculas, ¿lo aplico a estos archivos? o lo dejamos para otra vez?

Para la próxima, lo decía para discutirlo un poco no más ahora 😄

@rtobar rtobar merged commit 4776af7 into 3.10 Dec 11, 2021
@cmaureir cmaureir deleted the reduce_dicts branch December 12, 2021 09:26
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.

2 participants