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 object classification #298

Merged
merged 8 commits into from
Jun 20, 2024

Conversation

PawelLiberadzki
Copy link
Contributor

This PR introduces object classification in radar object tracking. User sets entity ids to object classes dictionary through dedicated API call. Object classes are available from object states, that are returned through public method of RadarTrackObjectsNode. Specifically, object class has to be parsed from classification probabilities.

Within this PR, radar object tracking tests were also be updated.

@PawelLiberadzki PawelLiberadzki requested a review from msz-rai June 10, 2024 11:10
@PawelLiberadzki PawelLiberadzki marked this pull request as ready for review June 10, 2024 11:19
Copy link
Collaborator

@msz-rai msz-rai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM.
Some minor comments to address.
I haven't approved it because there is a bug in the tape recording probably.

include/rgl/api/core.h Show resolved Hide resolved
include/rgl/api/core.h Show resolved Hide resolved
src/api/apiCore.cpp Outdated Show resolved Hide resolved
src/tape/TapeCore.hpp Show resolved Hide resolved
src/graph/RadarTrackObjectsNode.cpp Outdated Show resolved Hide resolved
@PawelLiberadzki PawelLiberadzki requested a review from msz-rai June 11, 2024 07:21
Copy link
Collaborator

@msz-rai msz-rai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

PawelLiberadzki and others added 7 commits June 17, 2024 16:13
Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Add handling nan radial speeds in radar postprocess and object tracking nodes.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
@msz-rai msz-rai force-pushed the feature/radar-object-classification branch from 0ee28c9 to bb367b1 Compare June 17, 2024 14:15
* Change array type in fieldData to be compatible with all RGL nodes

* Use velocities from raytrace instead of calculating it again (better accuracy)

* Do not output predicted objects

* Restore the conditions of merging detections into objects

* Fix required fields; skip test that fails

* Require detections in world frame to predict objects properly

* Fix displacementFromSkinning calculation

* Fix test

* Put all objects to the output

* Fix deltaTime calculation

* Fix test

* Fix time

* Fix passing time to new objects

* Add comment on skinning fix

* Fix coordinate system of width & length

* Fix bounding boxes

* Fix maxClassificationProbability

* Change unit of movement_sensitivity
@msz-rai msz-rai merged commit 5941780 into feature/q2-features Jun 20, 2024
20 checks passed
@msz-rai msz-rai deleted the feature/radar-object-classification branch June 20, 2024 13:53
msz-rai added a commit that referenced this pull request Jul 13, 2024
* Add initial support for object classification in RadarTrackObjectsNode.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Add API call for setting radar object classes.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Update radar track objects tests to handle object ids.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Add safety static_cast.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Make fixes based on the review.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Prevent underflow in loop condition (#308)

* Add handling nan radial speeds in radar nodes (#309)

Add handling nan radial speeds in radar postprocess and object tracking nodes.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Radar object classification improvements (#310)

* Change array type in fieldData to be compatible with all RGL nodes

* Use velocities from raytrace instead of calculating it again (better accuracy)

* Do not output predicted objects

* Restore the conditions of merging detections into objects

* Fix required fields; skip test that fails

* Require detections in world frame to predict objects properly

* Fix displacementFromSkinning calculation

* Fix test

* Put all objects to the output

* Fix deltaTime calculation

* Fix test

* Fix time

* Fix passing time to new objects

* Add comment on skinning fix

* Fix coordinate system of width & length

* Fix bounding boxes

* Fix maxClassificationProbability

* Change unit of movement_sensitivity

---------

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Co-authored-by: Maja Nagarnowicz <72594569+nebraszka@users.noreply.github.com>
Co-authored-by: Mateusz Szczygielski <112629916+msz-rai@users.noreply.github.com>
msz-rai added a commit that referenced this pull request Jul 15, 2024
* Add initial support for object classification in RadarTrackObjectsNode.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Add API call for setting radar object classes.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Update radar track objects tests to handle object ids.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Add safety static_cast.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Make fixes based on the review.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Prevent underflow in loop condition (#308)

* Add handling nan radial speeds in radar nodes (#309)

Add handling nan radial speeds in radar postprocess and object tracking nodes.

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>

* Radar object classification improvements (#310)

* Change array type in fieldData to be compatible with all RGL nodes

* Use velocities from raytrace instead of calculating it again (better accuracy)

* Do not output predicted objects

* Restore the conditions of merging detections into objects

* Fix required fields; skip test that fails

* Require detections in world frame to predict objects properly

* Fix displacementFromSkinning calculation

* Fix test

* Put all objects to the output

* Fix deltaTime calculation

* Fix test

* Fix time

* Fix passing time to new objects

* Add comment on skinning fix

* Fix coordinate system of width & length

* Fix bounding boxes

* Fix maxClassificationProbability

* Change unit of movement_sensitivity

---------

Signed-off-by: Paweł Liberadzki <pawel.liberadzki@robotec.ai>
Co-authored-by: Maja Nagarnowicz <72594569+nebraszka@users.noreply.github.com>
Co-authored-by: Mateusz Szczygielski <112629916+msz-rai@users.noreply.github.com>
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 this pull request may close these issues.

3 participants