Skip to content

feat: autograd support for rotated Box #2362

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

rahul-flex
Copy link
Contributor

@rahul-flex rahul-flex commented Apr 4, 2025

Autograd support for the Transformed class when the base geometry is a Box under rotation. Key changes are:

Rotated Normals:
Box face derivatives now accept rotation matrix, ensuring that the face derivative computed use rotated normals.

Derivative Computation:
The derivative with respect to rotated faces is computed using the surface mesh and gradient surfaces.

Testing:
Finite difference checks to validate autograd computations.
Emulated simulation test cases to verify rotation.
Test results attached for 7 cases: box, box with background structure, 0 deg rotation, 90 deg rotation about z, 45 deg about y, 45 deg about x, and 45 deg about z.
rel_error_center-x
rel_error_center-y
rel_error_center-z
rel_error_size-x
rel_error_size-y
rel_error_size-z

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant