Rename capacity_factors to capacity_factors_power
Closed this issue · 2 comments
Apparently capacity_factors (currently) only holds capacity_factors for the power sector ATM. This might be extended but I have no background knowledge on how likely and how soon that would happen, maybe @jacobvjk knows more.
While the status quo is as it is I would suggest renaming read_capacity_factors to read_capacity_factors_power. If doing so please make sure it does not interfere with the webtool though/adjust webtool accordingly.
Originally posted by @MirijaHa in #302 (comment)
at this point, the power sector is the only sector that will require capacity factors. We get capacity on the asset/company level and need to translate that information to power generation by applying the capacity factor. This happens so that we can calculate the profits on units sold. For all other sectors that we currently cover, the asset level unit already holds the same type of information as is needed in the profit calculation.
Which to me means that we can change the name but I do not see this as any priority, because I feel the likelihood to cause confusion as to which sectors are covered is low.