search

OSeMOSYS / OSeMOSYS_GNU_MathProg

The GNU MathProg implementation of OSeMOSYS
Apache License 2.0
9 stars 15 forks source link

Updated version date and changelog #9

Closed ResearchAtPlay closed 5 years ago

ResearchAtPlay commented 5 years ago

I updated the 1) version date and 2) changelog in "osemosys_short.txt" to reflect bugfix #8

willu47 commented 5 years ago

Hi @ResearchAtPlay - thanks for this. We've now added the historical changelog in docs/changelog.md, so could you instead add your entry to that file (see latest version on the master branch - you will either need to merge in the master to this branch, or rebase your branch on OSeMOSYS:master to get the latest version).

As part of the restructuring of the OSeMOSYS repository over the next few months, we will be moving towards the use of semantic versioning, and the version dates will no longer be used. In any case, the version of the code can now be identified by the commit hash. I'll be adding some guidance to make clear what the procedure is over the coming weeks.

If you'd like to get more involved, please see the issue tracker on OSeMOSYS/OSeMOSYS. Your contributions/comments/commits would be most welcome.

willu47 commented 5 years ago

Thanks @ResearchAtPlay - this looks good. Is this ready to merge into the master branch now?

ResearchAtPlay commented 5 years ago

Hi @willu47, I've reverted the version info update in osemosys_short.txt and instead updated ChangeLog.md to reflect the changes to accounting for CapitalCostStorage in the objective function. Go ahead and merge this with the master branch. Thanks!