oemof / DHNx

District heating system optimisation and simulation models
MIT License
27 stars 12 forks source link

Release: First release v0.0.1 #51

Closed jnnr closed 3 years ago

jnnr commented 3 years ago

Here we collect all open todos for the first release. The release process is documented here: https://github.com/oemof/oemof-solph/wiki/How-to-release

joroeder commented 3 years ago

Should the version __version__ = "0.0.1" also be added to __init__.py ?

jnnr commented 3 years ago

According to the practice in oemof-solph, the version should already be 0.0.1dev (https://github.com/oemof/oemof-solph/blob/dev/VERSION). We could introduce a new file VERSION and refer to it in setup.py.

In, init.py, we should add another version definition, as I read from the answer in https://github.com/oemof/oemof-solph/issues/712

joroeder commented 3 years ago

What should be the email adress in setup.py? For now, we could write the adresses: 'jann.launer@rl-institut.de, johannes.roeder@uni-bremen.de' (maybe this is the easiest solution for now), alternatively "contact@oemof.org" (however, there everything gets mixed), further alternative: create a new contact-dhnx@oemof.org.

jnnr commented 3 years ago

I vote for option 1 or 2.

jnnr commented 3 years ago

Shall we add an AUTHORS file?

joroeder commented 3 years ago

According to the practice in oemof-solph, the version should already be 0.0.1dev (https://github.com/oemof/oemof-solph/blob/dev/VERSION). We could introduce a new file VERSION and refer to it in setup.py.

Okay, but isn't the suggestion in oemof/oemof-solph#712 to get rid of the VERSION file? We don't have it yet, so everythings fine. For now, we can keep the '0.0.1' in setup.py for now (to change it to '0.0.1dev' to remove the 'dev' in the release again branch does not makes much sense to me). But we have to change it to '0.0.2dev' after the release, right?

In, init.py, we should add another version definition, as I read from the answer in oemof/oemof-solph#712

Okay, what do you mena by "other definition"? I think we also need to add the version '0.0.1' to the init.py, and change it to '0.0.2dev' after the release. As I understand oemof/oemof-solph#712, it is not nice to refer from the src to outside or the other way around.

In the end, as far I understand it, we will have the version in:

So we could ensure that they are the same for the first release manually. Or we use some of the tools like bumpversion - which you also need to check that everythings works probably. We could also add this tool later for the next releases.

joroeder commented 3 years ago

I vote for option 1 or 2.

Okay, then we go for 1. I don't know if I have access to the oemof contact emails. And if somebody wants to email a package specific question, the email is in the end forwarded to us. But this is not forever, we could change it with the next release again ...

joroeder commented 3 years ago

Shall we add an AUTHORS file?

I already added it in the features branch integrate_model_buil..._merge (I know probably not a good style)

jnnr commented 3 years ago

I tried to run it on a fresh environment. Scipy is missing in the requirements. Added a bullet point in the list on top.

joroeder commented 3 years ago

Thanks everybody!!!

The first release is done now! 🎆 🍾 🥳 🎉