MetaCell / MetaCell/NetPyNE-UI
[Proposal] Add warning message when setParam fails silently
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 23
- Fork
- 10
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
**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?"
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in netpyne/specs/netParams.py leggendo setParam e i metodi simili descritti nell’issue. Conferma il comportamento previsto quando manca un label, quindi verifica che venga emesso l’avviso e che il metodo continui a restituire False senza impostare il parametro.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100