NREL / ROSCO_toolbox

A repository for generating ROSCO controller input files and more.
http://rosco-toolbox.readthedocs.io/
Apache License 2.0
23 stars 20 forks source link

Refactor #22

Closed nikhar-abbas closed 3 years ago

nikhar-abbas commented 3 years ago

Refactor

Some large updates to re-factor a number of dependencies that the ROSCO toolbox has had thus far. The main motivation for this is to ease the use of ROSCO as a stand-alone tool.

Major changes

Minor Changes

To-Do list

Here's a list of items we need before this PR is ready to be merged. They can be added to the major and minor changes above as they get checked off.

This refactor will also necessitate changes to pCrunch and weis.

Some notes on possibilities:

nikhar-abbas commented 3 years ago

I removed the ROSCO submodule in b6295b3 and added a section to check for the conda-forge distribution of ROSCO in the setup.py file in 5689dbe.

dzalkind commented 3 years ago

There are some issues in the ROSCO_testing scripts. They want to use pCrunch and pCrunch wants to use ROSCO_toolbox utilities. Do we want to remove dependency on pCrunch while that is being revamped?

nikhar-abbas commented 3 years ago

There are some issues in the ROSCO_testing scripts. They want to use pCrunch and pCrunch wants to use ROSCO_toolbox utilities. Do we want to remove dependency on pCrunch while that is being revamped?

I think it might be smart to get a parallel pCrunch branch going to call the new ROSCO toolbox utilities, and eventually coordinate their merges into their respective develop/master branches.

dzalkind commented 3 years ago

This looks good. I'd like to merge it into develop. There is still one place with a pCrunch dependency, but I don't think it is used often: generate_rotperf_fast() in turbine.py.