-
Notifications
You must be signed in to change notification settings - Fork 19
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
msz-rai
merged 8 commits into
feature/q2-features
from
feature/radar-object-classification
Jun 20, 2024
Merged
Add radar object classification #298
msz-rai
merged 8 commits into
feature/q2-features
from
feature/radar-object-classification
Jun 20, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
msz-rai
reviewed
Jun 10, 2024
There was a problem hiding this 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.
msz-rai
approved these changes
Jun 11, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
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
force-pushed
the
feature/radar-object-classification
branch
from
June 17, 2024 14:15
0ee28c9
to
bb367b1
Compare
* 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
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.