[RTR] Fixing target plots

[EveCharbie]

All Submissions:

• Have you followed the guidelines in our Contributing document [docs/contribution.md]?
• Have you checked to ensure there aren't other open [Pull Requests] for the same update/change?
• Have you opened/linked the issue related to your pull request? Issue Plot errors in multi-phase #781
• Have you used the tag [WIP] for on-going changes, and removed it when the pull request was ready?
• When ready to merge, have you sent a comment pinging @pariterre in it?

New Feature Submissions:

1. Does your submission pass the tests (if not please explain why this is intended)?
2. Did you write a proper documentation (docstrings and ReadMe)
3. Have you linted your code locally prior to submission (using the command: black . -l120 --exclude "external/*")?

Changes to Core Features:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new examples for your core changes, as applicable?
• Have you written new tests for your core changes, as applicable?

---

This change is 

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 78.25%. Comparing base (e54be49) to head (7494252).
Report is 48 commits behind head on master.

Files	Patch %	Lines
bioptim/limits/penalty_option.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #851      +/-   ##
==========================================
- Coverage   78.32%   78.25%   -0.08%     
==========================================
  Files         140      139       -1     
  Lines       16232    16228       -4     
==========================================
- Hits        12714    12699      -15     
- Misses       3518     3529      +11     

Flag	Coverage Δ
unittests	78.25% <66.66%> (-0.08%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pariterre]

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @EveCharbie)

---

bioptim/limits/penalty.py line 97 at r1 (raw file):

                    to_first=list(range(controller.get_nlp.controls[key].cx_start.shape[0])),
                    to_second=list(range(controller.get_nlp.controls[key].cx_start.shape[0])),
                )  # TODO: why if condition, target_mapping not used (Pariterre?)

This is done elsewhere, isn't it?

[EveCharbie]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @pariterre)

---

bioptim/limits/penalty.py line 97 at r1 (raw file):

Previously, pariterre (Pariterre) wrote…

This is done elsewhere, isn't it?

Yes, what I just did fixed it.

[pariterre]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @EveCharbie)