search

astropy / astropy-APEs

A repository storing the Astropy Proposals for Enhancement.
Other
35 stars 37 forks source link

Clarifications to APE 0 #71

Closed Cadair closed 2 years ago

Cadair commented 2 years ago

Having run the first set of elections for Voting Members and for the Coordinating Committee the Governance Implementation Working Group noticed some rough edges and places where the meaning of APE 0 wasn't clear. This pull request makes a few (hopefully) small changes to the text to help those running future elections.

This PR is co-authored by @mwcraig and @ceb8 with input from @eteq.

hamogu commented 2 years ago

APE 0 says that changes to APE 0 need approval from 2/3 of the voting members. Since this is not terribly urgent, the Coco suggests to just vote on this the next time we have a voting members vote anyway. It's not long until we vote on adding voting members again, so we can just do that at the same time.

Cadair commented 2 years ago

My only comment on that would be that this modifies the process by which voting members are elected, with the aim of making that process clearer and easier for those running the next election. I know that this is obviously a trade off with dealing with voting on this APE, but worth considering.

Cadair commented 2 years ago

It's also not clear how the vote for the modifications should be run (might modify that language in a new commit), I would interpret that as requiring 2/3s of the voting members to approve the change, but there is no further indication on how that vote should proceed.

We could decide that 2/3s of the voting members should approve this pull request before we merge it.

mwcraig commented 2 years ago

We could decide that 2/3s of the voting members should approve this pull request before we merge it.

The proposal, after some discussion in the CoCo was to simply combine it with the new voting member vote that needs to be done soon (every 6 months, I think?). Not set in stone but it seemed like it would minimize the work needed to get it done.

It does read like it requires 2/3 of all voting members (not just those voting) for the changes to be approved.

adrn commented 2 years ago

These changes have been approved by the Astropy voting members in the latest Astropy election https://vote.heliosvoting.org/helios/elections/707e7c5c-fdf6-11ec-99a3-86fb037d8109/view

Cadair commented 2 years ago

Can this be merged then?

eteq commented 2 years ago

There was some discussion on changes above in the last few weeks after the vote occurred. So after discussion in the CoCo we agreed it makes sense to merge this as-is since the vote succeeded, but the two items above (https://github.com/astropy/astropy-APEs/pull/71#pullrequestreview-1048691995 and https://github.com/astropy/astropy-APEs/pull/71#pullrequestreview-824151602) will go in a follow-on change to be included in the next CoCo election.