Sooner-Rover-Team/auto_ros2

Try a slight refactor of `aruco_node`'s state

Ouverte

#145 ouverte le 17 nov. 2025

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Rust (7 forks)auto 404
good first issuesrc/aruco

Métriques du dépôt

Stars
 (6 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

it's nice to have this in its own file, but we should consider removing all the None just having users make the whole class None when no info has been found yet.

that's b/c we're trying to update it all at the same time, in a loop :)

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

Guide contributeur

Direction de recherche
Refactorez l'état de `aruco node` en rendant la classe entière optionnelle (par exemple en utilisant `Option<State>`) au lieu d'avoir de nombreux champs à `None`. Concentrez vous sur la mise à jour simultanée de tous les champs d'état dans une boucle.
Stack technique
rust
Domaine
autonomous driving
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75