Investment cost in NetRevenue.csv results output incorrect for retro generators

[ASamarkRoth]

If I understand correctly, generators with RETRO==1 should see investment costs according to Retro1_Inv_Cost_per_MWyr and not Inv_Cost_per_MWyr. This needs to be handled when writing NetRevenue.csv output: https://github.com/GenXProject/GenX/blob/5ccb3cab4b60c20fd40e45150a7f54133bb7304a/src/write_outputs/write_net_revenue.jl#L18

For users facing this issue, also applying Retro1_Inv_Cost_per_MWyr to Inv_Cost_per_MWyr would render expected costs in NetRevenue.csv. However, this will not work in the case of multiple sources for the retrofit with different values of Retro{n}_Inv_Cost_per_MWyr.

[cfe316]

Hi Anton,

Retrofits is currently an experimental feature, and should not have been included on the main branch; in fact we will gate it behind a warning that says so during the next update.

Thank you for raising these issues. If you have ideas on how to fix this, you are welcome to submit a pull request.

[ASamarkRoth]

Thanks for your reply!

Will keep this on our mind as we continue on our development!

[JesseJenkins]

@lbonaldo please confirm that the retrofit feature to be merged into develop (for v0.4 release) addresses this bug and if so, close issue.

[lbonaldo]

Hi Anton,

We've merged the new retrofit module (#600) which should address this issue. I've gone ahead and closed it, but please don't hesitate to reopen it if needed.

Thank you!