Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pull TimestampedVisionUpdate out of pose estimator #23

Merged
merged 1 commit into from
Feb 18, 2024
Merged

Pull TimestampedVisionUpdate out of pose estimator #23

merged 1 commit into from
Feb 18, 2024

Conversation

plusparth
Copy link
Member

Topic: refactor-vision-update
Relative: field-frame-estimator
Reviewers: saraansh, rithvik, pranav

@plusparth
Copy link
Member Author

plusparth commented Feb 17, 2024 •
edited
Loading

Reviews in this chain:
└#23 Pull TimestampedVisionUpdate out of pose estimator
　└#24 Use field frame estimator in drivetrain

@plusparth
Copy link
Member Author

plusparth commented Feb 17, 2024 •
edited
Loading

# head base diff date summary
0 d86e59a1 554383cb diff Feb 17 13:57 PM 5 files changed, 33 insertions(+), 31 deletions(-)
1 86087383 751a49d7 diff Feb 17 15:26 PM 0 files changed
2 39a8984c d3948cc5 diff Feb 17 19:02 PM 1 file changed, 2 insertions(+), 1 deletion(-)

@plusparth plusparth force-pushed the plusparth/revup/main/refactor-vision-update branch from e8d1250 to d86e59a Compare February 17, 2024 21:57
This was referenced Feb 17, 2024
Merged
Merged
Base automatically changed from plusparth/revup/main/field-frame-estimator to main February 17, 2024 23:25
@plusparth plusparth force-pushed the plusparth/revup/main/refactor-vision-update branch from d86e59a to 8608738 Compare February 17, 2024 23:26
@plusparth
Pull TimestampedVisionUpdate out of pose estimator
39a8984 
Topic: refactor-vision-update
Relative: field-frame-estimator
Reviewers: saraansh, rithvik, pranav
@plusparth plusparth force-pushed the plusparth/revup/main/refactor-vision-update branch from 8608738 to 39a8984 Compare February 18, 2024 03:02
@plusparth plusparth merged commit 63f2950 into main Feb 18, 2024
1 check passed
@plusparth plusparth deleted the plusparth/revup/main/refactor-vision-update branch February 18, 2024 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sswadkar sswadkar sswadkar approved these changes

@rbhog rbhog Awaiting requested review from rbhog

@AlphaPranav9102 AlphaPranav9102 Awaiting requested review from AlphaPranav9102

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@plusparth @sswadkar