Skip to content
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

matrix: fix slice to slice assignment to do deep copy #22578

Merged
merged 2 commits into from
Dec 22, 2023

Conversation

MaEtUgR
Copy link
Member

@MaEtUgR MaEtUgR commented Dec 21, 2023

Solved Problem

#22565 was reported on 1.14

Solution

Backport of #22574 for the point release 1.14.1

Changelog Entry

Bugfix: Position lock fix for non-default MPC_POS_MODE 0 (Direct velocity stick mapping)

To fix usage of a.xy() = b.xy() which should copy
the first two elements over into a and not act on a copy of a.
@MaEtUgR MaEtUgR added the v1.14 label Dec 21, 2023
@MaEtUgR MaEtUgR self-assigned this Dec 21, 2023
@MaEtUgR MaEtUgR merged commit 6eb8d04 into release/1.14 Dec 22, 2023
3 checks passed
@MaEtUgR MaEtUgR deleted the fix-matrix-slice-to-slice-assignment-1.14 branch December 22, 2023 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants