-
Notifications
You must be signed in to change notification settings - Fork 9
Add surface meshing parameters to Poisson surface reconstruction #60
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
Conversation
Co-authored-by: jf--- <620988+jf---@users.noreply.github.com>
Co-authored-by: jf--- <620988+jf---@users.noreply.github.com>
Co-authored-by: jf--- <620988+jf---@users.noreply.github.com>
Co-authored-by: jf--- <620988+jf---@users.noreply.github.com>
|
@copilot update the changelog.md |
Co-authored-by: jf--- <620988+jf---@users.noreply.github.com>
|
wait for merging of other PR's, than we need to rebase this PR |
|
Let's merge them one by one, so that there would be no conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
super clean, uses the Eigen::Ref<const compas::RowMatrixXd>.
@petrasvestartas this PR is not related to pending PR's so can be merged.
|
I guess updating the docs where changes in the reconstruction parameters are shown is relevant. |
|
i have integrated everything in the new mkdocs setup there is a bit more cleaning up to do, but getting there... |
wow, looks great! |
Summary
Successfully implemented support for additional parameter inputs for Poisson Surface Reconstruction:
Added Parameters:
Key Features:
Benefits:
Users can now control mesh quality and density to filter out vertices that don't belong to the original point cloud by adjusting sm_radius and sm_distance parameters, directly addressing the issue request.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.