[Proposal] Add warning message when setParam fails silently
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
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.
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
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?"
- Lingua principale
- JavaScript
- Stelle
- 23
- Fork
- 10
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
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.
Altre issue di MetaCell/NetPyNE-UI
-
Docker Build Error: Failed to Pull jupyter/base-notebook:hub-1.5.0 Due to Authorization Issue Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
MetaCell/NetPyNE-UI#832 ·
-
CNS24 Tutorial - Issues Aperta
MetaCell/NetPyNE-UI#829 · 2 commenti · 3 assegnatari ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
MetaCell/NetPyNE-UI#794 · 2 commenti ·
-
MetaCell/NetPyNE-UI#793 · 1 commento · 1 assegnatario ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
MetaCell/NetPyNE-UI#791 ·
Tutte le issue di MetaCell/NetPyNE-UI
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
palladius/rails8-app-on-gcp#145 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
dotenvx/dotenv-vscode#139 ·
-
test-change-proposal
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
web-platform-tests/interop#1455 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
corsairdev/corsair#1764 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100