MetaCell / MetaCell/NetPyNE-UI
[Proposal] Add warning message when setParam fails silently
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- JavaScript
- Estrelas
- 23
- Forks
- 10
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
**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**
```python
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:
```python
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?"
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece em netpyne/specs/netParams.py lendo setParam e os métodos semelhantes descritos na issue. Confirme o comportamento pretendido para um label ausente e, em seguida, verifique se o aviso é emitido e se o método ainda retorna False sem definir o parâmetro.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- backend
- Tipo de issue
- Funcionalidade
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 45/100