Skip to content

Fix: Dark Link's inputs in Mirror mode#3109

Merged
leggettc18 merged 1 commit intoHarbourMasters:develop-sulufrom
Archez:fix-dark-link-mirror
Aug 12, 2023
Merged

Fix: Dark Link's inputs in Mirror mode#3109
leggettc18 merged 1 commit intoHarbourMasters:develop-sulufrom
Archez:fix-dark-link-mirror

Conversation

@Archez
Copy link
Contributor

@Archez Archez commented Aug 8, 2023

Dark Link's update logic runs through Player_UpdateCommon just like the regular player. This means Dark Link's simulated "controller" inputs were getting inverted in mirror mode. This was causing him to move irregularly while mirror mode is active.

This adds another inverse to Dark Link's relative stick x input to negate the second inverse later on in Player_UpdateCommon, fixing his "AI" behavior when mirror mode is on.

Build Artifacts

Copy link
Contributor

@garrettjoecox garrettjoecox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🪞

Copy link
Contributor

@briaguya0 briaguya0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@leggettc18
Copy link
Contributor

Was merge ready removed on purpose? @garrettjoecox

@garrettjoecox
Copy link
Contributor

Was merge ready removed on purpose? @garrettjoecox

I removed it from the PRs that didn’t meet the timing criteria(en mass cause I added a bunch that didn’t meet the criteria).

In this case your approval makes it now meet the timing criteria (3 approvals and 3+ days since first approval) so re-adding the label

@leggettc18 leggettc18 merged commit 325c7fe into HarbourMasters:develop-sulu Aug 12, 2023
@Archez Archez deleted the fix-dark-link-mirror branch August 12, 2023 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants