Closed ppxasjsm closed 5 years ago
@davidlmobley could you have a look at this? The build shouldn't be failing and I have just rerun the Travis build and it worked fine.
Sorry for the delay, was tied up with grant deadlines. You mean it seems to be passing so you want to get it reviewed so we can merge?
I'm in the same place as @nividic at the moment so I'll see if he wants to have a look.
@nividic says he will look at it.
@nividic Any thoughts on this? I would really like to generate a release based on this. We can then happily move on by dissecting it completing and making very drastic changes, but it would be good to have this version set as something like 0.1. Let me know what you think.
@ppxasjsm In the end I think you have removed some of the tests e.g. graph_generation...are you going to put them back?
@nividic yeah I have added most of the tests back in. There is definitely room for more tests, but basically, I would like to just get a release of the current state since I used it in the D3R challenge, so I can reference the version in the writeup. Then we can go wild and plan changes for future updates etc.
Thinking of a 0.1 release or something along those lines, and then adhere to some pep8 version numbering convention as I doubt it would make sense for us to go with versioning including release years, as we are unlikely to release that often.
As for the conda install. This would be an added bonus, but can be done later? I would do a pypi release for sure and then people can install it with pip even in a conda environment. We have been having some nightmares with conda and resulting dependency updates, so I would like to spend a bit of careful time making sure the conda recipe really only includes the necessary dependencies.
The travis was passing. Not sure why it wasn't before. I made one type changed and pushed it and it should hopefully be fine?
Main features that were added: