Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Plan for Adding NatNet 4.2 Support

  • Update RigidBodyDescription NamedTuple to add optional quat field of type Optional[Vec4] after pos
  • Modify RigidBodyDescription.read_from_buffer to read the quaternion field when protocol version >= 4.2
  • Test the changes with protocol version 4.2 and earlier versions to ensure backward compatibility
  • Run linting tools (black) to ensure code style compliance
  • Run code review (no issues found)
  • Run security checks (no vulnerabilities found)
  • Remove .gitignore changes from PR as requested

Summary

Successfully implemented support for NatNet protocol version 4.2. The implementation adds an optional quaternion field to rigid body descriptions that is only read when the protocol version is 4.2 or higher, maintaining full backward compatibility with earlier versions.

Security Summary

No security vulnerabilities were discovered during the security scan.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: TimSchneider42 <25732590+TimSchneider42@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for NatNet 4.2 rigid body data Add NatNet 4.2 protocol support - quaternion field in rigid body descriptions Feb 6, 2026
Copilot AI requested a review from TimSchneider42 February 6, 2026 15:20
@TimSchneider42
Copy link
Owner

@copilot, remove the changes to the gitignore from this pr

Co-authored-by: TimSchneider42 <25732590+TimSchneider42@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 6, 2026

@copilot, remove the changes to the gitignore from this pr

Done in 353ee58

@TimSchneider42 TimSchneider42 marked this pull request as ready for review February 6, 2026 15:24
@TimSchneider42 TimSchneider42 merged commit f65eb18 into master Feb 6, 2026
1 check passed
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