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 dictionary input for setting aflr4AIM options. #236

Merged
merged 5 commits into from
Jul 28, 2023

Conversation

bburke38
Copy link
Contributor

Similar to recent updates in FUNtoFEM, this adds the ability to use dictionaries directly to set aflr4AIM options.

@bburke38 bburke38 added the enhancement New feature or request label Jul 25, 2023
@bburke38 bburke38 requested a review from timryanb July 25, 2023 15:49
tacs/caps2tacs/aflr_aim.py Outdated Show resolved Hide resolved
@sean-engelstad sean-engelstad linked an issue Jul 25, 2023 that may be closed by this pull request
@sean-engelstad
Copy link
Contributor

@bburke38 can you add this save_dict_options in for the TacsAim class too for consistency?

@timryanb
Copy link
Collaborator

timryanb commented Jul 27, 2023

@bburke38 can we use this new verbosity flag to silence the caps print outs in the integration tests here and here?

@gjkennedy gjkennedy merged commit 2ca603e into smdogroup:master Jul 28, 2023
@bburke38
Copy link
Contributor Author

@timryanb Yes, we should be able to use the verbosity flag to silence those print outs. Do you want me to make a new pull request just for those?

@timryanb
Copy link
Collaborator

@bburke38 yes, that would be great

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.

Constructor Flexibility in caps2tacs with tacsAIM
4 participants