-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Implement Adversarial Texture Attack on Object Trackers #1430
Implement Adversarial Texture Attack on Object Trackers #1430
Conversation
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
…ment_object_tracking_attack
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
…ment_object_tracking_attack
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
…ment_object_tracking_attack
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
…ment_object_tracking_attack
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Codecov Report
@@ Coverage Diff @@
## dev_1.9.0 #1430 +/- ##
=============================================
+ Coverage 81.55% 90.52% +8.96%
=============================================
Files 235 236 +1
Lines 19049 19214 +165
Branches 3365 3387 +22
=============================================
+ Hits 15535 17393 +1858
+ Misses 2807 1051 -1756
- Partials 707 770 +63
|
This pull request introduces 9 alerts when merging fb2db39 into 451d909 - view on LGTM.com new alerts:
|
This pull request introduces 9 alerts when merging a27d478 into 96ad148 - view on LGTM.com new alerts:
|
This pull request introduces 9 alerts when merging cfc743a into 3cdd1ae - view on LGTM.com new alerts:
|
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.
Hi Beat, many thanks for the review. I hope you find the comments helpful!
import torch # lgtm [py/repeated-import] | ||
|
||
y_pred = self._predictions(videos, y_init, foreground) | ||
loss = torch.nn.L1Loss(size_average=False)(y_pred[0]["boxes"].float(), target[0]["boxes"].float()) |
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.
size_average is deprecated, perhaps implement equivalent functionality using the reduction parameter
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 point, I'll replace it with reduction='sum'
.
from scripts.train import GoturnTrain | ||
from pathlib import Path | ||
|
||
_device = "cpu" |
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.
If a GPU is available, would it make sense to run the tests on a GPU to ensure that there is device consistency on this and future modifications of the code?
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.
That's good idea and would be very nice, at the moment there are no GPUs available on Actions. At the moment we run the tests manually on a GPU.
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
Signed-off-by: Beat Buesser <beat.buesser@ie.ibm.com>
This pull request introduces 9 alerts when merging 0f23952 into 3cdd1ae - view on LGTM.com new alerts:
|
Description
Continuing from PR #1353
This pull request implements the Adversarial Texture attack on object trackers by Wiyatno and Xu, 2019 (https://arxiv.org/abs/1904.11042) in PyTorch compatible with ART's PyTorchGoturn object tracking estimator. This implementation supports segmentation masks of foreground objects to train adversarial textures that are robust against being partially covered by the tracked or other objects.
Fixes #1326
Type of change
Please check all relevant options.
Checklist