Skip to content

Conversation

@cmichelenstrofer
Copy link
Contributor

@cmichelenstrofer cmichelenstrofer commented May 28, 2021

Update the ParaView pressureGlyphs.py macro for the latest ParaView version (5.9.1). The issue was that the current version of ParaView is not backwards compatible.

@kmruehl To Do:

  • Create new .pvsm files for the WEC-Sim_Applications/Paraview_Visualization cases using Paraview 5.9.1
  • Test new macro on WEC-Sim_Applications/Paraview_Visualization/RM3_MoorDyn_Viz. Currently only tested on WEC-Sim_Applications/Paraview_Visualization/OSWEC_NonLinear_Viz.
  • Mention the supported ParaView version somewhere in the Documentation or website.

@kmruehl kmruehl self-requested a review June 1, 2021 15:47
@kmruehl kmruehl added Bug bug in WEC-Sim source, high priority Visualization Visualization and Paraview labels Jun 1, 2021
@kmruehl
Copy link
Collaborator

kmruehl commented Jun 1, 2021

@cmichelenstrofer Thank you so much!

@kmruehl kmruehl linked an issue Jun 4, 2021 that may be closed by this pull request
@kmruehl
Copy link
Collaborator

kmruehl commented Jun 10, 2021

  • Update the doc to Paraview 5.9.1
  • Verify updates work with both WEC-Sim_Application cases
  • Add paraview docs for loading state file

TO DO: (On the WEC-Sim Applications repository:

  • Update *.pvsm state files for both WEC-Sim_Application cases with Paraview 5.9.1
  • Update README for WEC-Sim_Application cases to specify Paraview 5.9.1

@kmruehl kmruehl self-assigned this Jun 16, 2021
@kmruehl kmruehl merged commit 2a63daa into WEC-Sim:master Jul 2, 2021
@kmruehl
Copy link
Collaborator

kmruehl commented Jul 2, 2021

NOTE: I'm going to open a new PR on the wec-sim_applications repo to fix the PVSM state files. I'll tag this PR and #459

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug bug in WEC-Sim source, high priority Visualization Visualization and Paraview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setting up ParaView NL Hydro Visualization

2 participants