Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

RangeBearingSensor: line_style and poly_style are accepted but unused; animate the sensing beam and region

Offen
  • Stimmt etwas nicht?
#753 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
58/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
matplotlib, python
Bereich
data-visualization

Rechercherichtung

Start in sensors.py: the _line_style and _poly_style attributes are set in RangeBearingSensor.__init__, and plot() currently draws only the robot-to-landmark line. Read reading() to see where the robot's pose and the sensor's range and angle are available, then add a sector polygon and a beam line that are drawn or updated on each call using those styles. Done means the Agg-backend test shows both artists exist and change position after the robot moves, and the class docstring note from #750 is removed.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

tech-debt

RangeBearingSensor.__init__ accepts line_style and poly_style and stores them as _line_style and _poly_style, but nothing reads them. They were meant to style the sensor beam and the sensing region polygon, which should be drawn and updated every time reading() is called, based on the robot's current configuration (the toolbox's MATLAB ancestor did this).

At present plot=True only displays the map and draws a line from the robot to the landmark observed (sensors.py, plot()); the range limit and the angular field of view are never shown.

  • draw the sensing region (the sector between range and angle) as a polygon in poly_style, updated at each reading
  • draw the beam to the observed landmark in line_style
  • test with the Agg backend that the artists exist and move
  • remove the "accepted but not currently used" note from the class docstring (added in #750)

Deliberately left as is for now; this is the planned future work on animating the beam.

🤖 Generated with Claude Code

Vorherrschende Sprache
C++
Sterne
3.5k
Forks
639
Ø Merge
1 T. 16 Std.
Gemergte PRs (30 T.)
88

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus petercorke/robotics-toolbox-python

Alle Issues in petercorke/robotics-toolbox-python

Ähnliche Issues

Weitere Issues zu C++

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.