Skip to content

Conversation

rukawaaaaaaa
Copy link
Collaborator

The small deformation is replaced by fintie deformation. Now the beam can be defromed in any direction with the rotation large than pi/2

@rukawaaaaaaa rukawaaaaaaa requested a review from Xiangyu-Hu March 6, 2024 16:36
@DrChiZhang DrChiZhang added the enhancement New feature or request label Mar 6, 2024
Copy link
Owner

@Xiangyu-Hu Xiangyu-Hu left a comment

Choose a reason for hiding this comment

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

Could you put a new test case with regression data for the new features? Thanks.

dangular_vel_dt_[index_i] = getRotationFromPseudoNormalForSmallDeformation(
Vec3d(local_dpseudo_b_n_d2t), Vec3d(local_dpseudo_n_d2t), Vec3d(rotation_[index_i]), Vec3d(angular_vel_[index_i]), dt);
dangular_b_vel_dt_[index_i] = getRotationFromPseudoNormalForFiniteDeformation_b(
Vec3d(local_dpseudo_b_n_d2t), Vec3d(rotation_b_[index_i]), Vec3d(angular_b_vel_[index_i]), dt);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Here, Vec3d is not needed, since the slender structure simulation is always 3D.

Vec3d(local_dpseudo_b_n_d2t), Vec3d(local_dpseudo_n_d2t), Vec3d(rotation_[index_i]), Vec3d(angular_vel_[index_i]), dt);
dangular_b_vel_dt_[index_i] = getRotationFromPseudoNormalForFiniteDeformation_b(
Vec3d(local_dpseudo_b_n_d2t), Vec3d(rotation_b_[index_i]), Vec3d(angular_b_vel_[index_i]), dt);
dangular_vel_dt_[index_i] = getRotationFromPseudoNormalForFiniteDeformation(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please notice the writing, thanks:) Two more spaces here.

(rotation_1_b1 * rotation_1_b1 + rotation_1_b2 * rotation_1_b2 + Eps);

return Vec3d(angle_vel_dt_0, angle_vel_dt_1, 0.0);
return Vec3d(0, angle_vel_dt_1, 0.0);
Copy link
Collaborator

Choose a reason for hiding this comment

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

For writing consistency, 0.0 should be here.

2.0 * angular_b_vel[0] * angular_b_vel[2] * sin_rotation_b_2 * sin_rotation_b_0 * cos_rotation_b_0;

Real angle_b_vel_dt_2 = rotation_b_c * rotation_b_c * (rotation_b_c1 * cos_rotation_b_2 + rotation_b_c2 * sin_rotation_b_2) / (rotation_b_c1 * rotation_b_c1 + rotation_b_c2 * rotation_b_c2 + Eps);
return Vec3d(0, 0.0, angle_b_vel_dt_2);
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

Real rotation_0_b = sin_rotation_0 * angular_vel[0] * angular_vel[0] - dpseudo_n_d2t[1];
Real angle_vel_dt_0 = sin_rotation_0 * rotation_0_a + cos_rotation_0 * rotation_0_b;

*/
Copy link
Collaborator

Choose a reason for hiding this comment

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

These comments can be deleted now.

Vec3d(local_dpseudo_b_n_d2t), Vec3d(local_dpseudo_n_d2t), Vec3d(rotation_b_[index_i]), Vec3d(angular_b_vel_[index_i]), dt);
dangular_vel_dt_[index_i] = getRotationFromPseudoNormalForSmallDeformation(
Vec3d(local_dpseudo_b_n_d2t), Vec3d(local_dpseudo_n_d2t), Vec3d(rotation_[index_i]), Vec3d(angular_vel_[index_i]), dt);
dangular_b_vel_dt_[index_i] = getRotationFromPseudoNormalForFiniteDeformation_b(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since this pull request focuses on Finite Deformation/Rotation, could you add a test case with a large rotation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants