MetaCell / MetaCell/NetPyNE-UI
[Proposal] Add warning message when setParam fails silently
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 23
- Forks
- 10
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Description
While working on PEP8 cleanups in netpyne/specs/netParams.py, I noticed that setParam and similar methods fail silently when a label is not found.
Current Behavior
if label in self:
d = self[label]
else:
return False # Silently fails, user is not notified
Proposed Improvement
I propose adding an explicit warning so users can debug their scripts easier:
else:
print(f\"Warning: Label '{label}' not found. Parameter '{param}' not set.\")
return False
Question
Should I include this UX improvement as part of the PEP8 cleanup PR, or would you prefer a separate PR for this logic change?"
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Línea de trabajo
Comienza en netpyne/specs/netParams.py leyendo setParam y los métodos similares descritos en el issue. Confirma el comportamiento previsto para un label ausente y verifica después que se emite la advertencia y que el método sigue devolviendo False sin establecer el parámetro.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- backend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100