-
Notifications
You must be signed in to change notification settings - Fork 184
Fix paraview output function #1441
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
Conversation
|
@kmruehl I also amended our post-processing for the mooring class and the Paraview functions to account for multiple mooring connections. This should be working now with all data included in the output structure and the Paraview folder when turned on. One thing that is not set up is loading multiple mooring connections into Paraview. The |
6103046 to
4a9e413
Compare
|
To very this feature take the RM3 case and set |
|
@jtgrasb I'm getting the following error when I run the RM3_MoorDyn_Viz applications case with Index exceeds the number of array elements. Index must not exceed 161.
Error in writeParaviewMooring (line 60)
pt = [moorDyn.(['Line' num2str(iline)]).(['Node' num2str(inode) 'px'])(it), moorDyn.(['Line' num2str(iline)]).(['Node' num2str(inode) 'py'])(it), moorDyn.(['Line' num2str(iline)]).(['Node' num2str(inode) 'pz'])(it)];
Error in paraviewVisualization (line 62)
writeParaviewMooring(output.moorDyn, modelName, output.moorDyn.Lines.Time, datestr(simu.date), mooring.moorDynLines, mooring.moorDynNodes,simu.paraview.path,TimeBodyParav,NewTimeParaview)
Error in stopWecSim (line 38)
paraviewVisualization
Error in run (line 112)
evalin('caller', strcat(scriptStem, ';'));
Error in wecSim (line 56)
run('stopWecSim'); |
|
Disregard above error. I was working from In running this case I noticed and resolved another bug. We should be removing the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is ready for a merge once @jtgrasb confirms the remove vtk dir
|
Also added a couple variables to the clear statement to avoid errors if you run a model with Paraview twice in a row with different settings. Merging once tests pass. |
This PR addresses #1438 which found a bug in the paraview output function. If
simu.paraview.dtis larger thansimu.dt, the paraview timestep will only be applied to the first body. To fix this, I moved the if statement to before the bodies are looped through and moved the time vector selection outside of the if statement so it is applied for all bodies.Thanks for finding this issue and suggesting a fix @hamzaJayy.