Feature paraview

[agmoore4]

Several changes were made to improve visualization with ParaView and make WEC-Sim compatible with ParaView 5.11.1:

• Previously, the script WEC-Sim.py used BlockIndices with ExtractBlock. As of Paraview 5.10, ExtractBlock no longer works with BlockIndices. ExtractBlock was updated to use Selectors instead. Similarly, the method of counting the number of bodies had to be changed to be compatible with ParaView 5.11.1.
• Previously, the script pressureGlyphs.py could not find the properties it needed (Hydrostatic Pressure, etc.) because whitespace is automatically removed in calculator.Function strings. To resolve this, spaces were removed from the property names in writeParaviewBody.m (eg, HydrostaticPressure).
• The documentation for the ParaView section of user/advanced_features was updated to reflect the current examples in WEC-Sim_Applications. New animations for the two examples will be sent to Kelley to be included in the documentation.
• Previously, simu.paraview.startTime and simu.paraview.endTime were set to 0 and 100 by default in simulationClass.m, which led to the waves and the bodies to be visualized for different amounts of time. This was changed so that simu.paraview.startTime and simu.paraview.endTime are initialized as empty in simulationClass.m. Then in paraviewVisualization.m, if still empty after reading the input file, they are set to match simu.startTime and simu.endTime, respectively.
• Previously, the displayed times in ParaView were forced to always start at t=0 by subtracting off simu.paraview.startTime in paraviewVisualization.m. This was changed for clarity so that ParaView always refers to the beginning of the simulation as t=0, meaning that the visualization's first time step will always be the same value as simu.paraview.startTime.

[kmruehl]

Thanks @agmoore4 for your Paraview PR. My apologies for the delay reviewing it. Overall this looks really good. I had a comment regarding paraview.dt, otherwise I think this is ready for a merge.

[kmruehl]

Should paraview.dt = [ ] too? For consistency I think it should, and then it should also be updated in paraviewVisualization.m, similar to paraview.startTime and paraview.endTime

[kmruehl]

I think you should also add an if statement here to set simu.paraview.dt

[kmruehl]

I think all of this could be inside one if statement that re-assigns TimeBodyParaview only if paraview.startTime, paraview.endTime, or paraview.dt are specified. Otherwise all of these lines can be skipped, and go straight to line 41.

if isempty(simu.paraview.startTime) || isempty(simu.paraview.dt) || isempty(simu.paraview.endTime)
            if isempty(simu.paraview.startTime)
                simu.paraview.startTime = simu.startTime;
            end
            if isempty(simu.paraview.dt)
                simu.paraview.dt= simu.dt;
            end
            if isempty(simu.paraview.endTime)
                simu.paraview.endTime = simu.endTime;
            end
            NewTimeParaview(:,1) = simu.paraview.startTime:simu.paraview.dt:simu.paraview.endTime;
            PositionBodyParav = interp1(TimeBodyParav,PositionBodyParav,NewTimeParaview);
            TimeBodyParav = NewTimeParaview;
end

[agmoore4]

Implemented the requested changes.

[kmruehl]

@agmoore4 this looks good, and it's ready for a merge once you update the embedded videos. The update should be something like this for the RM3:

.. raw:: html

	<iframe width="560" height="315" src="https://www.youtube.com/embed/yL6LHdYTBIo" title="YouTube video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture; web-share" allowfullscreen></iframe>

and this for nonlinear:

.. raw:: html

	<iframe width="560" height="315" src="https://www.youtube.com/embed/JfKxQ1AgQBk" title="YouTube video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture; web-share" allowfullscreen></iframe>

[kmruehl]

Thanks @agmoore4. As soon as these tests run and pass I'll merge the PR

[kmruehl]

All tests are passing, so I'm going to merge this PR. Thanks again!