Sooner-Rover-Team/auto_ros2

Add new parameters for aruco search

Offen

#159 geöffnet am 20.02.2026

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (7 Forks)auto 404
configurationgood first issuesrc/navigator

Repository-Metriken

Stars
 (6 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We have constants in our basic search implementation of going in a circle. These should REALLY be parameters that we can adjust when we run the node. These constants are

  • SEARCH_RADIUS: the radius of the circle we want the Rover to drive around
  • SEARCH_POINTS: how many coordinates to generate (i.e. how many points should the circle of coordinates be made of)

This issue will involve editing the NavigationParameters type in src/navigator/navigator_node/types.py

Originally posted by @onkoe in https://github.com/Sooner-Rover-Team/auto_ros2/pull/156#discussion_r2831389483

Contributor Guide

Research-Richtung
Füge zwei neue Felder `search radius` und `search points` zum Typ `NavigationParameters` in `src/navigator/navigator node/types.py` hinzu. Aktualisiere dann die Suchfunktion, um diese Parameter anstelle der hartcodierten Konstanten zu verwenden. Stelle sicher, dass der Knoten diese Parameter aus dem ROS2 Parametersystem liest und an die Suchfunktion übergibt.
Tech Stack
python
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80