Skip to content

Conversation

@DamienGilliard
Copy link
Collaborator

@DamienGilliard DamienGilliard commented Jul 21, 2024

This PR concerns the last step of the process, where we identify the individual joints in a beam and propose an analysis of each one, through local registrations

1) assume two inputs: i_cloud and i_joints. i_cloud is the point cloud of the whole object, i_joints is a list of list the meshes of the joints. [[face_1_joint_A, face_2_joint_A, ...], [face_1_joint_B, face_2_joint_B, ...], ...]
2) segment the joints from the i_cloud and store them in a list that follows the same structure as i_joints: [joint_A, joint_B, ...]
3) for each joint point cloud, perform a registration by populating the corresponding mesh faces of each joint
4) apply the registration to the segmented joint point cloud and re-segment the joints
5) the result is a list of list of the segmented joints, where each segment is locally registered to the corresponding mesh faces: [[face_1_joint_A_arfter_reg_a, face_2_joint_A_after_reg_a, ...], [face_1_joint_B_after_reg_b, face_2_joint_B_after_reg_b, ...], ...]
--> that way we can analyse the joints individually (face 1 of joint A is too much to the left, ...)

@9and3 9and3 added enhancement New feature or request grasshopper backend and removed enhancement New feature or request labels Jul 22, 2024
@DamienGilliard
Copy link
Collaborator Author

ViewCapture20240725_135959
ezgif-1-3f1b1e8e58
UPDATE: in the current situation, the component isolates and merges the clusters of each joint, creates a reference point cloud of each joint, computes the transformation between both, and applies it. Therefore, the transformation applied to each joint is slightly different.

@DamienGilliard
Copy link
Collaborator Author

@9and3 , The branch is not yet fully ready for review, but close

@DamienGilliard
Copy link
Collaborator Author

image
We have a component !! This branch is ready for review @9and3 !! since it is almost only python stuff, I also include @eleniv3d

Copy link
Contributor

@9and3 9and3 left a comment

Choose a reason for hiding this comment

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

Thanks for this 🦖 ! Left few comments, the main one is about the synthax for the component, the rest seem good (also some questions) have a look!

Copy link
Contributor

@9and3 9and3 left a comment

Choose a reason for hiding this comment

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

noice!

@9and3 9and3 merged commit bcd6802 into main Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants