-
Notifications
You must be signed in to change notification settings - Fork 51
Minutes 2017 11 13
@orbeckst (email before meeting)
Let me briefly summarize what happened since the last call. This might be sufficient and we could skip the call on Monday, unless you want to discuss in person or if there are other developments to be shared.
-
Roadmap: https://github.com/alchemistry/alchemlyb/wiki/Roadmap was pretty much agreed upon (feel free to correct me or the roadmap)
-
Shuai did a lot of work!
- added Amber test case to alchemtest (merged https://github.com/alchemistry/alchemtest/pull/6)
- working Amber parser (open PR #32 https://github.com/alchemistry/alchemlyb/pull/32) but needs a few more tests for good test coverage
-
alchemtest: made licensing requirements for contributing test data explicit: we require an open license http://opendefinition.org/licenses/#recommended-conformant-licenses but effectively CC0 or CC-BY; see https://alchemtest.readthedocs.io/en/latest/ and https://github.com/alchemistry/alchemtest/pull/7 (approved by David and Michael)
-
decided to use logging (with Python logging library) throughout the library https://github.com/alchemistry/alchemlyb/issues/34 (allows calling code more control over what is output and makes it easy to capture output in a file)
-
decided to implement strict merging from PRs into master, i.e., everyone has to submit a PR in order to get code into the library. The code MUST be reviewed by a developer with write access and it MUST pass status checks: tests and testing coverage. See https://github.com/alchemistry/alchemlyb/wiki/Developer-Guide#pull-request-review-merge-model
Apologies if I missed anything, please add.
Doing the Amber parser was a good way to test and consolidate the development process and make some general decisions (licensing, strict merging). If you want to see the process in action, look at PR #32 https://github.com/alchemistry/alchemlyb/pull/32.
I think we are moving steadily forward with Amber, just needs a few more tests to get merged. We are relying on Michael for the Gromacs extended ensemble (https://github.com/alchemistry/alchemlyb/issues/14) but IIRC Michael said he would be super-busy until middle of November.
If we talk again in a month then it would be good to have the Amber parser fully functional, and initial steps taken towards the Gromacs XE parser, i.e., (1) tests in alchemtest and (2) an initial PR on alchemlyb that can be productively discussed.
Anything else?