GetMaterialCommand and GetTypologyCommand filter should be case insensitive
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Abre GetMaterialCommand alrededor de la línea 85 y localiza el filtro correspondiente en GetTypologyCommand. Revisa cómo cada comando compara su filtro con los nombres de materiales o tipologías y, después, verifica que las búsquedas devuelvan resultados independientemente de las mayúsculas y minúsculas en ambos comandos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The nature of the Materials library means that having to search for 'concrete' means having to run the search twice, for 'concrete' and 'Concrete' to find all relevant materials. I recommend that the filter is made case insensitive to allow such searches to find all relevant results with a single command.
e.g. line 85 of GetMaterialCommand could be
return materialObjects.Where(m => (m as IEnergyMaterialOpaque).Name.ToLower().Contains(command.Filter.ToLower())).ToList();
Please discuss if there are any use cases where case sensitivity is necessary.
- Lenguaje dominante
- Python
- Estrellas
- 3
- Forks
- 2
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de BHoM/LadybugTools_Toolkit
-
Queenbee path fails to resolve with default resolverPosiblemente ocupada @jamesramsden-bh la tomó hace 38 días. Abiertotype:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
BHoM/LadybugTools_Toolkit#361 ·
-
evaporative_cooling_potential_epw missing comma when calling the series version of the methodPosiblemente ocupada @Tom-Kingstone la tomó hace 3 días. Abiertotype:bug
BHoM/LadybugTools_Toolkit#364 · 1 asignado ·
-
Grasshopper Sample ScriptsQuizá libre de nuevo @Felix-Mallinder la tomó hace 85 días y no hay ningún pull request abierto. Abiertotype:feature
BHoM/LadybugTools_Toolkit#352 · 1 asignado ·
-
Upgrade pollination versionPosiblemente ocupada @Tom-Kingstone la tomó hace 135 días. Abiertotype:external-api-changes
BHoM/LadybugTools_Toolkit#349 · 1 asignado ·
-
Features from lbttkQuizá libre de nuevo @Tom-Kingstone la tomó hace 249 días y no hay ningún pull request abierto. Abiertosize:XL type:feature
BHoM/LadybugTools_Toolkit#335 · 1 asignado ·
Todos los issues de BHoM/LadybugTools_Toolkit
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
-
Validate finite output clamping limits in StepPredictor.prepare_clamping_params to prevent silent NaN/Inf propagationPosiblemente ocupada @muskan-creates352 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
mllam/neural-lam#848 ·
Los mantenedores suelen responder en 2 días
-
infra
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
fab-geocommuns/RNB-coeur#1166 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
deepspeedai/DeepSpeed#8951 ·
Los mantenedores suelen responder en 2 días