Skip to content

Conversation

dakejahl
Copy link
Contributor

See PX4/PX4-Autopilot#24153

The tiled meseeks images are necessary for optical flow to work since it relies on feature tracking.

@dakejahl dakejahl force-pushed the pr-x500_optical_flow_plugin branch from 6ee807a to e142d11 Compare February 14, 2025 23:27
@dakejahl dakejahl force-pushed the pr-x500_optical_flow_plugin branch from 7285a09 to f777ff1 Compare February 21, 2025 23:29
Copy link
Member

@Jaeyoung-Lim Jaeyoung-Lim left a comment

Choose a reason for hiding this comment

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

The image is 22MB. I dont think we want to have such large files in the git tree (at least in upstream)

@dakejahl
Copy link
Contributor Author

The image is 22MB. I dont think we want to have such large files in the git tree (at least in upstream)

I agree. That's why I'm really interested in the worlds in #78. Maybe we can cherry pick over the worlds/textures into this PR. I need to test the flow impl with those worlds as well as figure out if we can enable ground collision without performance issues

@dakejahl
Copy link
Contributor Author

dakejahl commented Feb 23, 2025

I added a new forest world which has a ground texture that works for optical flow
image

Copy link
Member

@Jaeyoung-Lim Jaeyoung-Lim left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@Jaeyoung-Lim Jaeyoung-Lim merged commit c15877d into PX4:main Feb 23, 2025
@hamishwillee
Copy link

Any docs updates needed?

@dakejahl
Copy link
Contributor Author

Any docs updates needed?

Yeah but will need to wait for PX4/PX4-Autopilot#24153

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.

3 participants