Make Vision pose estimation examples use all vision measurements#1706
Merged
samfreund merged 10 commits intoPhotonVision:mainfrom Apr 15, 2025
Merged
Make Vision pose estimation examples use all vision measurements#1706samfreund merged 10 commits intoPhotonVision:mainfrom
samfreund merged 10 commits intoPhotonVision:mainfrom
Conversation
Member
Author
|
Waiting on design decision before doing C++ and Python |
Contributor
|
Ship it |
2226037 to
179e315
Compare
spacey-sooty
commented
Feb 10, 2025
4aea314 to
9188f27
Compare
mcm001
reviewed
Apr 14, 2025
fb61129 to
89438a9
Compare
Resolves PhotonVision#1634 Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
Signed-off-by: Jade Turner <[email protected]>
89438a9 to
2c0d8ca
Compare
Contributor
|
Failing CI |
7 tasks
samfreund
approved these changes
Apr 15, 2025
samfreund
pushed a commit
that referenced
this pull request
Apr 15, 2025
## Description Updates the PhotonPoseEstimator programming documentation page to reflect the new PhotonPoseEstimator. and add a more comprehensive step-by-step tutorial using code from the PV Pose Estimation examples. Updates various code examples to ensure they are matching the latest documentation or example code on GitHub. This PR is a successor of, and therefore closes #1765. This PR is blocked by #1706, as the linked PR updates examples. Closes #1757 and closes #1800 and closes #1632 and closes #1773 and closes #1465. ## Meta Merge checklist: - [x] Pull Request title is [short, imperative summary](https://cbea.ms/git-commit/) of proposed changes - [x] The description documents the _what_ and _why_ - [x] If this PR changes behavior or adds a feature, user documentation is updated - [x] If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly - [x] If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1 - [x] If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated - [x] If this PR addresses a bug, a regression test for it is added --------- Co-authored-by: Kevin Reas <[email protected]> Co-authored-by: Matt Morley <[email protected]>
samfreund
added a commit
to samfreund/photonvision
that referenced
this pull request
Jun 28, 2025
…tonVision#1706) Resolves PhotonVision#1634 --------- Signed-off-by: Jade Turner <[email protected]> Co-authored-by: Sam Freund <[email protected]>
samfreund
pushed a commit
to samfreund/photonvision
that referenced
this pull request
Jun 28, 2025
## Description Updates the PhotonPoseEstimator programming documentation page to reflect the new PhotonPoseEstimator. and add a more comprehensive step-by-step tutorial using code from the PV Pose Estimation examples. Updates various code examples to ensure they are matching the latest documentation or example code on GitHub. This PR is a successor of, and therefore closes PhotonVision#1765. This PR is blocked by PhotonVision#1706, as the linked PR updates examples. Closes PhotonVision#1757 and closes PhotonVision#1800 and closes PhotonVision#1632 and closes PhotonVision#1773 and closes PhotonVision#1465. ## Meta Merge checklist: - [x] Pull Request title is [short, imperative summary](https://cbea.ms/git-commit/) of proposed changes - [x] The description documents the _what_ and _why_ - [x] If this PR changes behavior or adds a feature, user documentation is updated - [x] If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly - [x] If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1 - [x] If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated - [x] If this PR addresses a bug, a regression test for it is added --------- Co-authored-by: Kevin Reas <[email protected]> Co-authored-by: Matt Morley <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resolves #1634