USEPA / ElectricityLCI

Creative Commons Zero v1.0 Universal
24 stars 10 forks source link

FlowAmount column needs to be converted to float type to avoid error when performing groupby/aggregate #171

Closed jgrooviest closed 1 year ago

jgrooviest commented 2 years ago

https://github.com/USEPA/ElectricityLCI/blob/2232c41f2cb4fd333ad59c8710aa55906e6a7ed3/electricitylci/combinator.py#L200-L201

The sum aggregate function on FlowAmount will concatenate the string form of the flow amount, causing errors when converting the flow amount later (e.g., you get a cell value like 5.02125.02125). Converting to a float seems to fix the problem.