-
Notifications
You must be signed in to change notification settings - Fork 146
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
feat: add autoware_rosbag2_anonymizer usage #557
feat: add autoware_rosbag2_anonymizer usage #557
Conversation
Signed-off-by: Barış Zeren <bzeren1819@gmail.com>
Signed-off-by: Barış Zeren <bzeren1819@gmail.com>
…utoware-documentation into feat/add_rosbag2_anonymizer
Signed-off-by: Barış Zeren <bzeren1819@gmail.com>
I will test this tool this week. |
@StepTurtle
|
Thanks @mitsudome-r for your reply, can you share output of the |
@StepTurtle
|
Since the error was around supervision, I manually installed groundingdino, open_clip_torch, and ultralytics, then reinstalled supervision 0.22.0 manually. |
Fixed with this commit: autowarefoundation/autoware_rosbag2_anonymizer@9445da4
|
Signed-off-by: Barış Zeren <bzeren1819@gmail.com>
H, @mitsudome-r I think there are no more comment or request for this PR. Is there anything we should wait for? |
@StepTurtle Thanks for the ping. I'm doing another test to make sure that there are no issues left. |
Signed-off-by: Barış Zeren <baris@leodrive.ai>
Signed-off-by: Barış Zeren <baris@leodrive.ai>
Signed-off-by: Barış Zeren <baris@leodrive.ai>
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.
LGTM!
Description
This PR updates the dataset documentation and adds instructions for the data anonymizer tool.
Related issue:
Repo Link:
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.