Skip to content

Quitando palabras repetidas en cada diccionario#1699

Merged
rtobar merged 2 commits into3.10from
reduce_dicts
Dec 11, 2021
Merged

Quitando palabras repetidas en cada diccionario#1699
rtobar merged 2 commits into3.10from
reduce_dicts

Conversation

@cmaureir
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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