NREL / reV

Renewable Energy Potential (reV) Model
https://nrel.github.io/reV/
BSD 3-Clause "New" or "Revised" License
107 stars 24 forks source link

Supply-Curve-Aggreation: Take partial exclusions into account for data_layer characterization sums #261

Closed WilliamsTravis closed 3 years ago

WilliamsTravis commented 3 years ago

Bug Description In supply-curve-aggregation, the "sum" characterization method for data_layers appears to be adding the pixel counts within a supply curve point before removing partial exclusions. We would like this figure to allow us to calculate the area of non-excluded land within each characterization.

Expected behavior The sum of a characterization layer is a float taking into account partial exclusions.

System (please complete the following information):

MRossol commented 3 years ago

@WilliamsTravis is this what you were envisioning: https://github.com/NREL/reV/blob/77c77a065b6be7704eae09c13e1e964ddaa289e5/reV/supply_curve/point_summary.py#L472-L473