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

Add radar sensor #466

Closed
Lukasnol opened this issue Nov 6, 2024 · 3 comments · Fixed by #522
Closed

Add radar sensor #466

Lukasnol opened this issue Nov 6, 2024 · 3 comments · Fixed by #522
Assignees

Comments

@Lukasnol
Copy link
Collaborator

Lukasnol commented Nov 6, 2024

Detailed Description

create radar node.

Definition of Done

  • see above

Effort Estimate

No response

Testability

No response

Dependencies

No response

@ll7
Copy link
Member

ll7 commented Nov 7, 2024

Do you need a node to add a sensor, or do you need to adapt the sensor config file?

@Toni2go Toni2go added the p1 label Nov 7, 2024
@Toni2go
Copy link
Collaborator

Toni2go commented Nov 7, 2024

By "sensor config file" do you mean the file paf/code/agent/src/agent/agent.py? I think the radar sensor is already configured there, right?
The idea was to create a node similar to lidar_distance.py, so in this node we would subscribe to the RADAR topic and only publish the data important to us like distance, velocity, ...

@ll7
Copy link
Member

ll7 commented Nov 8, 2024

Okay, so you are thinking about a radar measurement processing node. Yes, that would be helpful. 👍

@Ralf524 Ralf524 self-assigned this Nov 12, 2024
@RoyaLxPole RoyaLxPole added this to the Sprint 02 2024-11-25 milestone Nov 12, 2024
@michalal7 michalal7 self-assigned this Nov 20, 2024
@SirMDA SirMDA removed the p1 label Nov 23, 2024
@Ralf524 Ralf524 linked a pull request Nov 24, 2024 that will close this issue
7 tasks
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in PAF Project Backlog Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

7 participants