please remove duplicate value check from node-red-node-pisrf
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- embedded-iot
Piste de recherche
Ouvrez node-red-nodes/hardware/PiSrf/nrsrf.py autour des lignes 83–85 et examinez comment la vérification des valeurs en double affecte les mesures publiées. Confirmez le comportement attendu pour les distances valides comprises entre 2 et 400, puis vérifiez que les valeurs valides répétées sont publiées tandis que les valeurs hors plage restent exclues.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Which node are you reporting an issue on?
node-red-node-pisrf
What are the steps to reproduce?
use it
What happens?
duplicate values are dropped
What do you expect to happen?
every value comes through
Please tell us about your environment:
- [ 3.0.2] Node-RED version:
- node.js version:
- npm version:
- Platform/OS:
- Browser:
pisrf sensors are prone to noise (esp. if there are more present), so error correction is necessary anyway (f.e. with median). This gets difficult if the message count isn't consistent. The duplicate check (duplicate values don't get published) is kind of poor and can be easily implemented with a filter node. On the other hand, a median filter f.e. is difficult when the message count is not predictable.
Just change:
if distance != OLD and distance > 2 and distance < 400:
print(distance)
OLD = distance
to
if distance > 2 and distance < 400:
print(distance)
in node-red-nodes/hardware/PiSrf/nrsrf.py at line 83 to 85 or at least make it an option. I am not familiar with pull requests, otherwise i would submit one.
Thank you
ps: i don't know why the new lines in the code brackets get swallowed, i tried, normally it does work as expected
- Langage dominant
- JavaScript
- Étoiles
- 1.1k
- Forks
- 610
- Merge moyen
- 1 j 8 h
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
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.
Autres issues de node-red/node-red-nodes
-
Difficulté 3/5 Une demi-journée Accessibilité débutants 42/100
node-red/node-red-nodes#1152 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
node-red/node-red-nodes#1151 · 4 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 58/100
node-red/node-red-nodes#1142 · 11 commentaires ·
-
XMPP node issue (follow #1135)Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
node-red/node-red-nodes#1141 · 15 commentaires ·
-
email-in node crashed node-redOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
node-red/node-red-nodes#1130 · 4 commentaires ·
Toutes les issues de node-red/node-red-nodes
Issues similaires
-
bug documentation
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
awslabs/visual-asset-management-system#880 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
[BUG]: Workspace settings forms stay at 50% width on narrow screens and wrap text word by wordOuvertepossible bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Mintplex-Labs/anything-llm#6889 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
swc-project/swc-next#1165 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
nhsuk/nhsuk-prototype-kit-package#439 ·
Les mainteneurs répondent en général sous 1 jour