Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unify camera detections into one node #126

Open
danielrhuynh opened this issue Jun 14, 2024 · 2 comments · May be fixed by #104
Open

Unify camera detections into one node #126

danielrhuynh opened this issue Jun 14, 2024 · 2 comments · May be fixed by #104
Assignees

Comments

@danielrhuynh
Copy link
Contributor

Instead of our three nodes for camera detection, we want one unified node that outputs one message with detections for resource efficiency, and temporal synchronization, or we could have a post processing node that applies a time synchronizer using http://wiki.ros.org/message_filters

June 13 2024

  • Running in foxglove with post-processing synchronization (?)
  • Still testing post-process temporal synchronization
  • Still working on getting SLURM + foxglove working on all machines
@danielrhuynh
Copy link
Contributor Author

June 27 (Skipped a wear for midterms)

  • Done!
  • Need to fix linter for PR
  • We're going with this implementation because the solutions cleaner and it's more resource efficient

@danielrhuynh
Copy link
Contributor Author

July 25 2024

  • Waiting for review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants