X Tutup
Skip to content

FIX: AxisComposite not respecting min/maxValue (case 1335838).#1331

Merged
Rene-Damm merged 3 commits intodevelopfrom
fix-axis-composite-ignoring-minmax
Jul 5, 2021
Merged

FIX: AxisComposite not respecting min/maxValue (case 1335838).#1331
Rene-Damm merged 3 commits intodevelopfrom
fix-axis-composite-ignoring-minmax

Conversation

@Rene-Damm
Copy link
Contributor

@Rene-Damm Rene-Damm commented May 17, 2021

Fixes 1335838 (FogBugz).

Description

AxisComposite.ReadValue didn't actually do much with minValue and maxValue. My assumption is this got lost when the method was changed to respect control actuation.

Changes made

Made negative push away from midPoint towards minValue and made positive push away from midPoint towards maxValue.

Checklist

Before review:

  • Changelog entry added.
    • Explains the change in Changed, Fixed, Added sections.
    • For API change contains an example snippet and/or migration example.
    • FogBugz ticket attached, example ([case %number%](https://issuetracker.unity3d.com/issues/...)).
    • FogBugz is marked as "Resolved" with next release version correctly set.
  • Tests added/changed, if applicable.
    • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.

During merge:

  • Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

@Rene-Damm Rene-Damm requested a review from andrew-oc May 17, 2021 11:42
@Rene-Damm Rene-Damm merged commit 38bdeec into develop Jul 5, 2021
@Rene-Damm Rene-Damm deleted the fix-axis-composite-ignoring-minmax branch July 5, 2021 18:59
bryandube added a commit that referenced this pull request Jul 19, 2021
* Switch default controller for unmatched layouts to XRControllerWithRumble and support '_' in sanitized strings

* reverted rumble change

* reverted changelog message since change was reverted

* FIX: Action reference destroyed in player prefab (case 1319756, #1316).

* FIX: Fixing compilation of tests on 2021.1 and short-term remediating docfx issues (#1322)

* Added input action asset validation message to PlayerInputManager and editor (#1297)

* Fix for tests

* Pin windows CI image to an exact version for now (#1323)

Authored-by: Dmytro Ivanov <dmytro@unity3d.com>

* NEW: Device Simulator plugin for Input System (#1264)

* FIX: Events queued during OnUpdate being lost (case 1297339). (#1321)

Co-authored-by: Rene Damm <rene@unity3d.com>

* CHANGE: InputSystemUIInputModule defaults are set via Reset instead of default script references (fixes 1323566) (#1317)

* FIX: Correctly selecting dropdown items when going up through hierarchy (#1309)

* NEW: Adding UNITY_INCLUDE_TESTS define constraint to package test assemblies (#1294)

* FIX: Fixing Keyboard.current becoming null after OnScreenButton is disabled/destroyed (#1310)

* FIX: clickCount not incremented correctly (case 1317239, #1315).

* CHANGE: Add event processing limits (#1325)

* DOCS: Update LICENSE and CONTRIBUTIONS.

* RELEASE: 1.1-pre.4.

* FIX: Package validation failure.

* FIX: Tests failing from change to -pre.

* FIX: InputUser.pairedDevices getting corrupted (case 1327628, #1326).

* FIX: Misaligned binding paths when in text mode (case 1200107, #1328).

* FIX: Calling InputSystem.Update from inside action callback causes stack overflow (case 1316000) (#1327)

* Added InputSystem.runUpdatesInEditMode for XR support (#1319)

* Fixing CI (#1357)

* FIX: InputTestFixture leaving .current at null (case 1329015, #1345).

* FIX: GetBindingDisplyString() doubling up output for composite bindings (case 1321175, #1330).

* FIX: Console errors for PlayerInput with empty action maps (case 1317735, #1341).

* FIX: Broken script references in Touch Samples (case 1190598, #1339).

* FIX: PointerInput in TouchSamples registered too late (case 1215048, #1342).

* FIX: MultiTapInteraction not respecting multiTapDelayTime setting (case 1292754, #1337).

* MERGE: stable => develop (#1343).

* FIX: Looking for wrong control types when rebinding parts of composites (case 1272563, #1338).

* FIX: No RMB menu when clicking empty tree views (case 1336426, #1351).

* FIX: EnumerateChangedControls skipping PS4 dpad left/right/down (case 1315107, #1336).

* FIX: AxisComposite not respecting min/maxValue (case 1335838, #1331).

* FIX: Undo not working in input settings (case 1291709, #1332).

* FIX: Remove error message when PlayerInput can't find action (case 1259577, #1334).

* CHANGE: InputDevice.OnConfigurationChanged now virtual (#1329).

* FIX: ArgumentOutOfRangeException in IsPointerOverGameObject (case 1337354, #1356).

* FIX: Stuck HoldInteraction when held&released in same event (case 1346786, #1361).

* FIX: Incorrect array indexing in InputUser. (#1362)

* FIX: Incorrect indexing in InputUser.OnDeviceChanged that could result in incorrect pairing of devices.

* FIX: Incorrect indexing when sorting in InputActionRebindingExtensions.RebindingOperation (#1364)

* FIX: Incorrect indexing when sorting magnitude based on score in InputActionRebindingExtensions.RebindingOperation.

* FIX: 'Add Action' not resetting interactions and processors (#1369).

* CHANGE: Touch OnPointerExit now deferred one frame (case 1347048, #1365).

* FIX: Setting motor speeds and light bar color in quick succession on DualShock 4 (#1346)

* FIX: Setting motor speeds and light bar color in quick succession on DualShock 4

This commit fixes the issue with calling SetMotorSpeeds followed by SetLightBarColor in quick succession on a DualShock 4 controller. The IOCTL interface currently only allows a single command at a time to be processed on the input device, so the second of these calls will usually fail. The fix contained here is simply to expose an extra method on the DualShock 4 interface called SetMotorSpeedsAndLightBarColor that combines both instructions into a single IOCTL command, but a more robust fix will be added later that deals with this issue universally at a lower level in the InputDevice hierarchy.

* Add documentation and tweaked changelog

* FIX: Callback ordering issues when adding/removing during execution (case 1322530, #1335).

* FIX: PlayerInputManager join action not triggering (#1344)

* FIX: Submit and cancel UI actions now trigger on press instead of release (#1363)

* FIX: InputControlPathDrawer showing duplicate entries (#1366)

* FIX: Broken link in CONTRIBUTIONS.md (#1367)

* FIX: WebGL gamepad up/down controls were not being inverted (#1370)

* Add XboxOne/DualShock layouts for Android (#1255)

* ADDED: DualShock4GamepadAndroid and XboxOneGamepadAndroid layout for Android

* FIX: Right stick to use AXIS.Z and AXIS.RZ for Android gamepads.

* FIX: Fixed triggers to always use Axis.Gas and Axis.Brake for Android gamepads.

Co-authored-by: Rene Damm <rene@unity3d.com>
Co-authored-by: andrew-oc <78356434+andrew-oc@users.noreply.github.com>
Co-authored-by: Andrew O'Connor <andrew.oconnor@unity3d.com>

* Updated changelog to match after merge

* Revert "Updated changelog to match after merge"

This reverts commit f83bc75.

* Revert "Merge branch 'develop' into xr-layout-builder-openxr-fixes"

This reverts commit 3037847, reversing
changes made to b87964c.

Co-authored-by: Rene Damm <rene@unity3d.com>
Co-authored-by: Dmytro Ivanov <jimon.j1m0n@gmail.com>
Co-authored-by: andrew-oc <78356434+andrew-oc@users.noreply.github.com>
Co-authored-by: Roman Osipov <31279599+RomanO-Unity@users.noreply.github.com>
Co-authored-by: Tom Baird <tomb@unity3d.com>
Co-authored-by: David Tetlow <65656964+DTetlow@users.noreply.github.com>
Co-authored-by: Håkan Sidenvall <hakan.sidenvall@unity3d.com>
Co-authored-by: Brendan Duncan <brendanduncan@gmail.com>
Co-authored-by: rytis-buz <52201119+rytis-buz@users.noreply.github.com>
Co-authored-by: Andrew O'Connor <andrew.oconnor@unity3d.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup