Remove in-place operations

[beat-buesser]

Description

This pull request removes in-place operations that overwrite the input data.

Fixes #2262

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 #2263 (52f64a8) into dev_1.15.2 (8d9de16) will not change coverage.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

@@             Coverage Diff             @@
##           dev_1.15.2    #2263   +/-   ##
===========================================
  Coverage       85.62%   85.62%           
===========================================
  Files             308      308           
  Lines           27476    27476           
  Branches         5048     5048           
===========================================
  Hits            23526    23526           
  Misses           2669     2669           
  Partials         1281     1281           

Files Changed	Coverage Δ
...mators/object_detection/pytorch_object_detector.py	84.65% <100.00%> (ø)
art/estimators/object_detection/pytorch_yolo.py	82.48% <100.00%> (ø)

[f4str]

Hi @beat-buesser thank you fixing this issue! I've tested it with the attack_adversarial_patch_pytorch_yolo.ipynb demo notebook and it is working once again without issues.

One minor comment that the change in translate_predictions_xcycwh_to_x1y1x2y2 is not needed, but functionality-wise its the same. Feel free to keep the change or revert it back.

[f4str]

This isn't necessary since labels is a newly created tensor. But the functionality is still the same.