Save the applied added mass time series #1023
Conversation
Remove 'fixed' mass option from OSWEC input file
nathanmtom
left a comment
There was a problem hiding this comment.
@akeeste the first change file is related to updates made by a recent PR so no concerns. The modification to the postProcessWECSim is only adding an additional call during execution and this should not influence dynamic performance.
My only comment is that we should add some documentation in the developer section to illustrate how this postProcessing is being used to correct the addedmass force time history. Currently the advanced features developer section only describes the addedMass adjustment process. This does not need to be implemented in this PR but flagging as this book keeping is important to document to reduce future confusion.
|
@akeeste Thanks for this PR, I approve and will let you merge when ready. |
|
@nathanmtom Thanks for your quick review. I removed the extraneous file and added a project card for myself to expand the developer docs on our post processing corrections. |
This PR adds a call in
postProcess.mto the pre-existingbodyClass.storeForceAddedMass()function. This allows WEC-Sim to save the timeseries of the added mass force that is applied during the simulation.This is necessary for a TEAMER award where the added mass force is varied during simulation. I'd like to change a couple other items in the added mass post-processing, but this is the least intrusive and most important for visualizing the current TEAMER work.