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

ekf2: fix optical_flow_vel publication #22260

Merged
merged 1 commit into from
Oct 25, 2023
Merged

ekf2: fix optical_flow_vel publication #22260

merged 1 commit into from
Oct 25, 2023

Conversation

bresch
Copy link
Member

@bresch bresch commented Oct 25, 2023

We should otherwise call this publication before the aid_src publisher that sets the timestamp. Having it separate avoids this ordering constraint.

ekf2: fix optical_flow_vel publication
cf7f3d8 
We should otherwise call this publication before the aid_src publisher
that sets the timestamp. Having it separate avoids this ordering
constraint.
@bresch bresch added the EKF2 label Oct 25, 2023
@bresch bresch requested a review from dagar October 25, 2023 13:17
@bresch bresch self-assigned this Oct 25, 2023
dagar
dagar approved these changes Oct 25, 2023
View reviewed changes
Copy link
Member

@dagar dagar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could all go away if/when we move it to simply publish directly in the backend.

@dagar dagar merged commit eed2870 into main Oct 25, 2023
87 checks passed
@dagar dagar deleted the pr-flow-vel-pub branch October 25, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dagar dagar dagar approved these changes

Assignees

@bresch bresch

Labels
EKF2
Projects
PX4 EKF
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bresch @dagar