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

Change and Verify mass.adjoint = False #285

Open
sean-engelstad opened this issue Feb 27, 2024 · 0 comments
Open

Change and Verify mass.adjoint = False #285

sean-engelstad opened this issue Feb 27, 2024 · 0 comments

Comments

@sean-engelstad
Copy link
Contributor

sean-engelstad commented Feb 27, 2024

  • change default mass Function classmethod to have func.adjoint = False
  • Verify that this doesn't mess up the derivatives in FUN3D using the function.id attributes and pushing and pulling composite function calls. The function.id values are setup in the Scenario,Body class
  • Fails in the tacs_interface.py with the following when there were 4 total functions, and mass was implemented as adjoint=False. Seems like some of these shape_term expressions should include adjoint_functions as the mass does have structural shape derivatives. (so in the setup need to change the size)
  File "/p/home/seanfire/git/funtofem/funtofem/driver/_funtofem_driver.py", line 315, in _extract_coordinate_derivatives
    solver.get_coordinate_derivatives(scenario, self.model.bodies, step)
  File "/p/home/seanfire/git/funtofem/funtofem/interface/tacs_interface.py", line 963, in get_coordinate_derivatives
    struct_shape_term[:, ifunc] += array.astype(body.dtype)
IndexError: index 3 is out of bounds for axis 1 with size 3

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

No branches or pull requests

1 participant