Closed samgdotson closed 1 year ago
Thanks for the review @abachma2. There might be some repetition since I was working on this and the previous PR simultaneously. Fortunately there are no code conflicts (per github). I pulled in the previous merge so it should be okay.
Thanks for the review @LukeSeifert, if you're satisfied you can go ahead and merge. I updated the doc string to reflect the change in the default value.
@abachma2 @LukeSeifert @yardasol @munkm I renamed the two ambiguous functions objective_from_X
. Hopefully, this satisfies everyone. I also added equations to the docstrings.
Nudge @abachma2 @yardasol
Please review PR #38 before reviewing this one!
Updates
This PR adds several things
per_unit_energy
orper_unit_capacity
)and it updates the
capacity_expansion
module to account for generic objectives.