Add general object detection estimator for PyTorch

[beat-buesser]

Description

This pull request adds a general, model-independent, object detection estimator for PyTorch models in art.estimators.object_detection.PyTorchObjectDetector.

Fixes #1196

Type of change

Please check all relevant options.

• Improvement (non-breaking)
• Bug fix (non-breaking)
• New feature (non-breaking)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

[codecov-commenter]

Codecov Report

Merging #1295 (173481f) into dev_1.8.0 (58839f7) will increase coverage by 0.43%.
The diff coverage is 60.57%.

@@              Coverage Diff              @@
##           dev_1.8.0    #1295      +/-   ##
=============================================
+ Coverage      80.90%   81.34%   +0.43%     
=============================================
  Files            226      227       +1     
  Lines          19284    19300      +16     
  Branches        3499     3502       +3     
=============================================
+ Hits           15602    15699      +97     
+ Misses          2379     2304      -75     
+ Partials        1303     1297       -6     

Impacted Files	Coverage Δ
...ted_gradient_descent/projected_gradient_descent.py	94.44% <ø> (ø)
art/utils.py	74.08% <ø> (+10.70%)	⬆️
...imators/object_detection/python_object_detector.py	59.76% <59.76%> (ø)
...estimators/object_detection/pytorch_faster_rcnn.py	90.90% <80.00%> (+31.51%)	⬆️
art/estimators/object_detection/__init__.py	100.00% <100.00%> (ø)
art/attacks/evasion/brendel_bethge.py	84.93% <0.00%> (+0.32%)	⬆️
art/defences/detector/poison/activation_defence.py	85.18% <0.00%> (+0.33%)	⬆️
...rt/defences/detector/poison/clustering_analyzer.py	97.46% <0.00%> (+2.53%)	⬆️
art/estimators/poison_mitigation/strip/strip.py	77.08% <0.00%> (+4.16%)	⬆️
... and 4 more

[lgtm-com]

This pull request introduces 4 alerts and fixes 1 when merging 8f64dd3 into 07e18ee - view on LGTM.com

new alerts:

• 2 for Module is imported more than once
• 2 for Unused import

fixed alerts:

• 1 for Module is imported more than once

[lgtm-com]

This pull request introduces 4 alerts and fixes 1 when merging 8f64dd3 into bb5b5c1 - view on LGTM.com

new alerts:

• 2 for Module is imported more than once
• 2 for Unused import

fixed alerts:

• 1 for Module is imported more than once

[lgtm-com]

This pull request introduces 2 alerts and fixes 1 when merging 5579ce6 into bb5b5c1 - view on LGTM.com

new alerts:

• 2 for Module is imported more than once

fixed alerts:

• 1 for Module is imported more than once

[lgtm-com]

This pull request introduces 1 alert and fixes 3 when merging 66ce420 into bb5b5c1 - view on LGTM.com

new alerts:

• 1 for Module is imported more than once

fixed alerts:

• 3 for Module is imported more than once

[GiulioZizzo]

Hi Beat, thank you for the review and I have left a few comments!

[GiulioZizzo]

does the type hint for x need to be updated as we check several times if x is either an array or a tensor? (similar for later hints as well)

Perhaps also the name and description could be updated as we do not compute the loss here

[beat-buesser]

For now we don't support torch.Tensor for x and checked in line 182, but I agree with you for future releases.

[lgtm-com]

This pull request introduces 1 alert and fixes 3 when merging 8271c09 into 56a9fad - view on LGTM.com

new alerts:

• 1 for Module is imported more than once

fixed alerts:

• 3 for Module is imported more than once

[lgtm-com]

This pull request introduces 1 alert and fixes 3 when merging eb1f1cf into 58839f7 - view on LGTM.com

new alerts:

• 1 for Module is imported more than once

fixed alerts:

• 3 for Module is imported more than once

[lgtm-com]

This pull request introduces 1 alert and fixes 3 when merging 173481f into 58839f7 - view on LGTM.com

new alerts:

• 1 for Module is imported more than once

fixed alerts:

• 3 for Module is imported more than once