openforcefield / standards

A repository of the standards employed across the Open Force Field Consortium.
https://openforcefield.github.io/standards
MIT License
1 stars 3 forks source link

OFF-EP 0000 #1

Closed SimonBoothroyd closed 3 years ago

SimonBoothroyd commented 3 years ago

Description

This PR tracks the initial draft of this repositories contents. I opted to include the main documents in this PR rather than the main initial commit so as to avoid putting very WIP standards and procedures front and center from the start.

See https://github.com/openforcefield/openff-toolkit/issues/741 for more details for the rationale behind this repo + PR.

cc @j-wags @mattwthompson @Yoshanuikabundi

Status

mattwthompson commented 3 years ago

I've fleshed out a few more sections and tinkered with some details (in ways that may be personal preference and not consensus).

A major outstanding question from my perspective is how to constitute technical committees

j-wags commented 3 years ago

One overarching OpenFF committee OR one for each relevant domain

Let's initially just assign the OpenFF governing board as the uber-committee, and let them decide the number/content of the subcommittees.

mattwthompson commented 3 years ago

@davidlmobley - I've come around @j-wags's suggestion of starting with simple criteria, but doing so with the understanding that they may change once we try them out in practice. If you were confused about murky criteria for accepting EPs, you're not alone - the lack of concretely documented processes for PEPs becoming accepted seem to be intentional, and there's sometimes confusion from the community about who has the "final say" in any. (I'm not sure about the analogs for NumPy and conda-forge EPs.) So I think it's fair to not set our goals too lofty here but instead be flexible for future changes.

I think the path forward here to get approvalfrom the 4 current SMIRNOFF committee members (@j-wags, @davidlmobley , @SimonBoothroyd , @jchodera from https://github.com/openforcefield/openff-toolkit/issues/741#issuecomment-783601422) that this document is a workable starting point and this PR should be merged. From there, we can clarify as needed some of the remaining details (the roles of the governing board and other other committees, processes for OFF-EPs being approved, etc.) as changes to this EP or new EPs. This would also let us act on updates to the SMIRNOFF spec, which was discussed several months ago as the rationale for all of this. This is somewhat timely as we want clearly-documented spec updates to help with some corner cases that have been plaguing the toolkit lately (https://github.com/openforcefield/openff-toolkit/issues/716, https://github.com/openforcefield/openff-toolkit/issues/882, https://github.com/openforcefield/openff-toolkit/issues/734, probably more coming down the pipeline with other infrastructure work and some users doing experiments with non-bonded functional forms) and better ways of incorporating feedback in general.

mattwthompson commented 3 years ago

Tracking approval here:

SimonBoothroyd commented 3 years ago

Because OFF-EPs may also track procedural or community based changes I've added @karmencj to the approvers list and modified the acceptance criteria to a simple majority. The main rationale is that not all of the approvers will need to be involved in all OFF-EPs (e.g. I'm sure @davidlmobley and @karmencj will not be particularly interested in certain software specific OFF-EPs.)