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.

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

Fix fast_floor in FPC Filter, Invalid PLY file (expected 'property uint8 views') #1716

Merged
merged 2 commits into from
Nov 5, 2023

Conversation

pierotofy
Copy link
Member

@pierotofy pierotofy commented Nov 4, 2023

Fixes a fault in FPC Filter which was causing the program to stall forever in certain reconstructions when using --pc-sample , caused by a bad floor implementation.

@pierotofy
Copy link
Member Author

pierotofy commented Nov 5, 2023

Also fixes the Invalid PLY file (expected 'property uint8 views') error which several people have encountered, due to a fault in ODM's calls to OpenMVS's subscene handling.

@pierotofy pierotofy changed the title Fix fast_floor in FPC Filter Fix fast_floor in FPC Filter, Fix Invalid PLY file (expected 'property uint8 views') Nov 5, 2023
@pierotofy pierotofy changed the title Fix fast_floor in FPC Filter, Fix Invalid PLY file (expected 'property uint8 views') Fix fast_floor in FPC Filter, Invalid PLY file (expected 'property uint8 views') Nov 5, 2023
@Saijin-Naib
Copy link
Contributor

Do you think this is worthy of a new release of Windows? I think this is a nice fix up.

@pierotofy
Copy link
Member Author

Probably yes.

@pierotofy pierotofy merged commit 2df4afa into OpenDroneMap:master Nov 5, 2023
2 checks 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