DOCS: DOCATT-8754: Updated TrackedDevice class documentation#2298
DOCS: DOCATT-8754: Updated TrackedDevice class documentation#2298
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
🤖 Helpful? Please react with 👍/👎 | Questions❓Please reach out in Slack #ask-u-pr-agent |
PR Code Suggestions ✨Explore these optional code suggestions:
🤖 Helpful? Please react with 👍/👎 | Questions❓Please reach out in Slack #ask-u-pr-agent |
|||||||||
Codecov ReportAll modified and coverable lines are covered by tests ✅ @@ Coverage Diff @@
## develop #2298 +/- ##
===========================================
- Coverage 77.95% 77.94% -0.02%
===========================================
Files 477 476 -1
Lines 97419 97405 -14
===========================================
- Hits 75943 75922 -21
- Misses 21476 21483 +7 Flags with carried forward coverage won't be shown. Click here to find out more.
... and 5 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
chris-massie
left a comment
There was a problem hiding this comment.
I gave a suggestion to replace a Pose link to go to https://docs.unity3d.com/Packages/com.unity.xr.openxr@1.16/manual/input.html#pose-data instead of the Tracked Pose Driver component documentation since it's more direct to what the pose data is.
Packages/com.unity.inputsystem/InputSystem/Devices/TrackedDevice.cs
Outdated
Show resolved
Hide resolved
- Fixed some links, clarified isTracked return in TrackedDevice.cs - Added related XR packages to `xref` list in projectMetadata.json
…ce.cs Co-authored-by: Chris Massie <67029035+chris-massie@users.noreply.github.com>
b1b6f12 to
3c70988
Compare
Description
Added documentation for the
InputSystem.TrackedDeviceclass, particularly for the 4 properties. Several users had requested more information about these properties and their values, specifically theIntegerControlvalue in thetrackingStateproperty.Testing status & QA
No code was altered; documentation changes only. However, I did run the formatting tool and regenerated the documentation locally to test it.
Overall Product Risks
Very low risk (documentation only).
Comments to reviewers
I've requested a technical review from Chris Massie, who kindly provided me with some background on this class.
Checklist
DOCS: ___.