When outputting power sector data with consumed rates using the new warnings, the warnings caught that there were some missing generated rates in the outputs. After looking into this, it looks like this was happening because we were implementing .fillna(0) before we replaced inf values with na.
This PR moves the fillna after the inf replacement, and also explicitly only fills na values that were a result of net_generation_mwh (the denominator in the generated rate calculation) being zero. This means that the expected behavior should be:
If there is zero net generation but positive emissions mass, the reported emission rate should be zero instead of NA.
If there is some other reason why the generated rate is missing, these missing values should be left so that they are flagged by our validation checks, since this is unexpected behavior.
When outputting power sector data with consumed rates using the new warnings, the warnings caught that there were some missing generated rates in the outputs. After looking into this, it looks like this was happening because we were implementing
.fillna(0)before we replacedinfvalues withna.This PR moves the fillna after the
infreplacement, and also explicitly only fills na values that were a result ofnet_generation_mwh(the denominator in the generated rate calculation) being zero. This means that the expected behavior should be: