Nate deVelder

This is a minor comment, but we should probably make sure either/both the spack package and the local repo openfast package are set up for version 3.5.* before we merge?...

Maybe this is backward, but don't we want the tagged release _before_ changing the api, so that there's a last good point for the older versions of openfast that we...

I might be confusing this more...apologies...I just wanted to have a way to specify the latest old openfast compatible version without having to look up the commit sha @psakievich is...

One of our team members just had an issue compiling your PR branch. `[ 60%] Building CXX object CMakeFiles/amrwind_obj.dir/amr-wind/wind_energy/actuator/disk/disk_ops.cpp.o /projects/wind_advcontrol/gyalla/src/amr-wind/amr-wind/wind_energy/actuator/turbine/fast/FastIface.cpp: In instantiation of 'void exw_fast::{anonymous}::fast_func(const FType&&, Args ...) [with FType...

Haha, yeah I was just counting out arguments and it's the extra int, ok, I'll let him know. Sorry that was a false issue

I'm going to close this for now so as not to clutter the issue list...I don't believe this is important enough for anyone at the moment, and for the type...

Started looking at this. Note that "monin_obukhov_averaging_type" options are only "none" or "planar" and "none" is the local option (and just doesn't pick up anything from boundary layer stats). My...

@lawrenceccheung maybe we should also try a precursor with increasing refinement levels turned on? I think we wanted to do one of those for the benchmark anyway? I could imagine...

And if it wasn't clear, the x-axis on the Nalu-Wind portion of the second plot is not right...these need more work to incorporate the number of eqn system iterations

 Here it is fixed