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

35v2 objective stress rates #66

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

saifr68
Copy link
Collaborator

@saifr68 saifr68 commented Jan 29, 2025

Objective stress rate is implemented and a test fie is written. Plots and paraview output files are created for the ease of review. Can be removed later

Copy link
Collaborator

@aradermacher aradermacher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice


if self.mesh_update:

# Update to current configuration
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this code is nearly identically with the one in self.build_history (factor 0.5). Would it make sense to put that in an separate class function? And do we need to do the interpolation twice?


mesh_update = True
co_rotation = True

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please add a small comment part describing the test and its purpose

@aradermacher
Copy link
Collaborator

The test are not yet working. Please have a look @saifr68

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.

2 participants