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.

Sign up for GitHub

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

Jump to bottom

[ROS2] Feature: Improved Lidar Result retrieval and publishing #736

Closed
alek-kam-robotec-ai opened this issue Jul 25, 2024 · 0 comments · Fixed by #751
Closed

[ROS2] Feature: Improved Lidar Result retrieval and publishing #736

alek-kam-robotec-ai opened this issue Jul 25, 2024 · 0 comments · Fixed by #751
Labels
priority/major Major priority. Work that should be handled after all blocking and critical work is done. sig/simulation Categorizes an issue or PR as relevant to SIG Simulation

Comments

@alek-kam-robotec-ai
Copy link
Contributor

alek-kam-robotec-ai commented Jul 25, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.
This issue relates to the growing complexity of the Lidar result data. The current implementation of both lidar result retrieval and publishing is outdated and broken (see #719). It also makes it hard to add new, optional features (see #713 and #737).

Describe the solution you'd like
A new design of the raycast results handling should be designed and implemented.
@alek-kam-robotec-ai alek-kam-robotec-ai added needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Jul 25, 2024
@alek-kam-robotec-ai alek-kam-robotec-ai mentioned this issue Jul 25, 2024
Merged
@jhanca-robotecai jhanca-robotecai added sig/simulation Categorizes an issue or PR as relevant to SIG Simulation and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels Jul 25, 2024
This was referenced Aug 7, 2024
Closed
Merged
Merged
Closed
@alek-kam-robotec-ai alek-kam-robotec-ai mentioned this issue Aug 22, 2024
Merged
@michalpelka michalpelka added priority/major Major priority. Work that should be handled after all blocking and critical work is done. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority/major Major priority. Work that should be handled after all blocking and critical work is done. sig/simulation Categorizes an issue or PR as relevant to SIG Simulation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ROS2] Raycast Results Refactor RobotecAI/o3de-extras
3 participants
@michalpelka @alek-kam-robotec-ai @jhanca-robotecai