Skip to content

Conversation

@akaszynski
Copy link
Collaborator

@akaszynski akaszynski commented May 7, 2025

Current cyclic writer does support writing out expanded cyclic results. This PR adds that feature and also merges the individual sectors by default. User can override this behavior by setting expand_cyclic=False.

Example:

Plot the Z component of the displacement for the first mode at the 2nd nodal diameter.

# using academic rotor from test_data
filename = 'result.vtk'
academic_rotor.save_as_vtk(filename)

import pyvista as pv
grid = pv.read(filename)
grid.plot(scalars="Nodal solution (0, -2)", component=2)

image

Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM!

@RobPasMue RobPasMue enabled auto-merge (squash) May 8, 2025 06:55
@RobPasMue RobPasMue merged commit 251f692 into main May 8, 2025
13 checks passed
@RobPasMue RobPasMue deleted the feat/cyclic-save-vtk branch May 8, 2025 06:58
@akaszynski
Copy link
Collaborator Author

@RobPasMue could we do a patch release for this?

@RobPasMue
Copy link
Member

Sure, on it!

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.

3 participants