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

Add more enum values to ConfigureResolveAutoInertials #1371

Open
scpeters opened this issue Feb 4, 2024 · 0 comments
Open

Add more enum values to ConfigureResolveAutoInertials #1371

scpeters opened this issue Feb 4, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@scpeters
Copy link
Member

scpeters commented Feb 4, 2024

Desired behavior

From #1335 (comment):

Regarding //inertial/@auto, the behavior was supposed to be determined by ConfigureResolveAutoInertials, but I don't think it's working correctly because it's exporting all inertia values unconditionally.

I think the logic should be the following in pseudocode

auto==true && SAVE_CALCULATION:
  export all inertia
auto==true && not SAVE_CALCULATION:
 export `//inertial/auto`

But it looks like we need more enum values for "skip calculation during load and do not save calculation" and "calculate during load but do not save calculation".

Alternatives considered

Implementation suggestion

Additional context

@scpeters scpeters added the enhancement New feature or request label Feb 4, 2024
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
Status: To do
Development

No branches or pull requests

1 participant