Issue 113 merge private GridConnectionCost into public GridConnectionCost module

WISDEM / LandBOSSE

New LandBOSSE Model (Developed in 2018-2019)

Other

17 stars 25 forks source link

Issue 113 merge private GridConnectionCost into public GridConnectionCost module #114

Closed parangat94 closed 4 years ago

parangat94 commented 4 years ago

@akey7 @eberlea ready for review.

akey7 commented 4 years ago

@parangat94 I just reviewed these changes, and the validation data passes. So go ahead and hold off on splitting the lines up for now. We can go ahead with the code the way it is. Just keep the comments I made earlier as potential thoughts for the future, in case they are helpful.