dzalkind

Results 65 comments of dzalkind

Following our in-person code review today, I think we are okay to merge this once tests pass. I updated the AEP TI calculation here: https://github.com/WISDEM/WEIS/blob/eeeae1076c149aec70a8247e46a393976d88b867/weis/dlc_driver/dlc_generator.py#L410 Some future wish list items:...

I think this issue has been resolved with the fix in WISDEM. If not, please re-open and let us know how we can help.

@gbarter, what kind of instructions should we include? I've seen other cases (ROSCO) where Windows users are required to conda install `mpi4py`.

I added a note to this effect in the `docs` branch. I think the original issue has been resolve, too. Thanks!

Hi Athul, Thanks for adding this. It looks great so far. Do you need to merge in the latest ctopt? There seem to be a lot of extra changes besides...

Hi Athul, Overall, this is looking great. I have it working locally with the latest `ct-opt` branch with some minor tweaks. It runs really fast as expected for a controller...

Let's continue the push on DFSM over here: https://github.com/NREL/WEIS-CTOpt/pull/2 Once that's in good shape, we can merge it back here for FOWTs.

@mayankchetan, is this a WEIS issue or something that should be addressed in OpenFAST?

There is a `model_only` flag in the modeling options that should skip the simulations: https://github.com/WISDEM/WEIS/blob/77512f2244be4e445425b107c4c40d67326cfaed/weis/inputs/modeling_schema.yaml#L39 https://github.com/WISDEM/WEIS/blob/77512f2244be4e445425b107c4c40d67326cfaed/weis/aeroelasticse/openmdao_openfast.py#L610

Here is where the BTS files are generated: https://github.com/WISDEM/WEIS/blob/77512f2244be4e445425b107c4c40d67326cfaed/weis/aeroelasticse/openmdao_openfast.py#L1821 It should skip that. My personal workaround would be to only make one short BTS file, [like in this example](https://github.com/WISDEM/WEIS/blob/77512f2244be4e445425b107c4c40d67326cfaed/examples/02_run_openfast_cases/modeling_options.yaml#L39). That...