Sooner-Rover-Team/auto_ros2

Try a slight refactor of `aruco_node`'s state

Aberta

#145 aberto em 17 de nov. de 2025

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Rust (7 forks)auto 404
good first issuesrc/aruco

Métricas do repositório

Stars
 (6 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Refatore o estado do `aruco node` tornando a classe inteira opcional (por exemplo, usando `Option<State>`) em vez de ter muitos campos como `None`. Foque em atualizar todos os campos de estado simultaneamente em um loop.
Pilha de tecnologia
rust
Domain
autonomous driving
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75