please remove duplicate value check from node-red-node-pisrf
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- embedded-iot
Rechercherichtung
Öffne node-red-nodes/hardware/PiSrf/nrsrf.py im Bereich der Zeilen 83–85 und untersuche, wie sich die Prüfung auf doppelte Werte auf veröffentlichte Messwerte auswirkt. Bestätige das gewünschte Verhalten für gültige Entfernungen zwischen 2 und 400 und überprüfe anschließend, dass wiederholt auftretende gültige Werte veröffentlicht werden, während Werte außerhalb des Bereichs weiterhin ausgeschlossen bleiben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.1k
- Forks
- 610
- Ø Merge
- 1 T. 8 Std.
- Gemergte PRs (30 T.)
- 1
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus node-red/node-red-nodes
-
Schwierigkeit 3/5 Ein halber Tag Anfängerfreundlichkeit 42/100
node-red/node-red-nodes#1152 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
node-red/node-red-nodes#1151 · 4 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 58/100
node-red/node-red-nodes#1142 · 11 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
node-red/node-red-nodes#1141 · 15 Kommentare ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
node-red/node-red-nodes#1130 · 4 Kommentare ·
Alle Issues in node-red/node-red-nodes
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
bigcalendar/react-big-calendar#2823 ·
-
feature t-tooling
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
neutral:fix
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
documentation enhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100
OWASP/cve-lite-cli#1480 ·
Maintainer antworten meist innerhalb von 2 Tagen