MetaCell / MetaCell/NetPyNE-UI
[Proposal] Add warning message when setParam fails silently
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 23
- Forks
- 10
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
**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?"
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans netpyne/specs/netParams.py par lire setParam et les méthodes similaires décrites dans l’issue. Confirmez le comportement attendu lorsqu’un label est absent, puis vérifiez que l’avertissement est émis et que la méthode renvoie toujours False sans définir le paramètre.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100