MetaCell / MetaCell/NetPyNE-UI

[Proposal] Add warning message when setParam fails silently

Aberta
#835 0 comentários 0 reações 0 responsáveis Ver no GitHub

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

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. 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

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.