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

Update cylinder flow SVD tutorial to SmartSim 0.8.0. #32

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

Conversation

AndreWeiner
Copy link
Contributor

The latest SmartSim version does not allow duplicated model names within a single Experiment. Moreover, each model creates a new subfolder corresponding to the model name when the model is executed. The subfolder creation can be avoided by passing the path argument to Experiment.create_model(). Naming conflicts arising when computing the SVD with different values for the rank truncation parameter are avoided by including the rank parameter in the model name.

Partially resolves #29.

The latested SmartSim version does not allow for duplicated model
names within a single Experiment. Moreover, each model creates a
new subfolder corresponding to the model name when the model is
executed. The subfolder creation can be avoided by passing a the
*path* agrument to Experiment.create_model(). Naming conflicts
arising when computing the SVD with different values for the rank
truncation parameter are now avoided by including the rank parameter
in the model name.
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

Successfully merging this pull request may close these issues.

Update to SmartSim 0.8
1 participant