Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add initial organization charter and project governance #2367

Merged
merged 9 commits into from
Sep 6, 2022

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Aug 23, 2022

This PR sets up the initial governance structure for the vispy project and the vispy organization. It is based on GitHub's Minimum Viable Governance project. I haven't filled in any VisPy specific information or added any specific members to any groups. I'm hoping people will volunteer for one or both of:

  1. Steering Committee
  2. Maintainers

I guess I could list most of the people I consider active maintainers in the project and let you tell me if/when you don't want to be in that list before we merge this.

@djhoese djhoese marked this pull request as ready for review August 24, 2022 01:24
Comment on lines 12 to 20
| **NAME** | **Organization** |
|-----------------|------------------|
| Lorenzo Gaisfas | N/A |
| David Hoese | N/A |
| Almar Klein | N/A |
| Eric Larson | N/A |
| Kai Mühlbauer | N/A |
| Cyrille Rossant | N/A |
| Nicolas Rougier | N/A |
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brisvag @almarklein @larsoner @kmuehlbauer @rossant @rougier I've listed all of you as maintainers to the library. If you think I've missed someone let me know. If you'd like to not be on this list, also let me know.

As you can read in the GOVERNANCE.md document, being a maintainer now doesn't mean anything different than it did before. The general idea with this governance structure is that it is all about "good faith" consensus. We can do things to this project if we think it goes along with the other maintainers' opinions/desires. The only thing that requires explicit consensus from a certain amount of maintainers are changes to the governance document or if anything is disagreed with.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove me at this point, I'm not sure my vision of the project is clear enough anymore to make important decisions :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @larsoner. I've removed you from the maintainers list. I'm sure we'll still bug you when we can.

Comment on lines +14 to +19
| **NAME** | **Handle** | **Affiliated Organization** |
|-----------------|-------------|-----------------------------|
| David Hoese | @djhoese | N/A |
| Almar Klein | @almarklein | N/A |
| Cyrille Rossant | @rossant | N/A |
| Nicolas Rougier | @rougier | N/A |
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brisvag @almarklein @larsoner @kmuehlbauer @rossant @rougier I've listed me, Almar, Cyrille, and Nicolas as the members of the steering committee. The role of the committee is described in the CHARTER.md document. The only reason I didn't list everyone is that it seemed like it would make decision making harder. I don't feel that strongly about it so let me know what you think. We could always set up some kind of rotating membership of an active steering committee and inactive members. Or we could do that type of thing with the vispy project maintainers.

Thoughts?

Copy link
Collaborator

@brisvag brisvag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rougier
Copy link
Contributor

rougier commented Aug 29, 2022

Thanks for putting this up. I'm not sure to fully understand the antitrust policy because from what I've read it looks like it allows you to do pretty much anything. Probably I read it wrong. Also, is there a precise reason this is necessary?

@rougier
Copy link
Contributor

rougier commented Aug 29, 2022

Same question for trademarks: is it necessary? I guess it doesn't hurt to have it explicit but I wonder the origin of such section: does that means that some projects got problem because they did not have such section?

@djhoese
Copy link
Member Author

djhoese commented Aug 29, 2022

@rougier Thank you for bringing up these questions. You are right on the antitrust, it basically says do whatever you want. I think the main benefit to having it here is that it defines it if it ever comes up in the future. While for most people who are used to the open source scientific python ecosystem would just assume this, I don't think it hurts to state it explicitly. For the trademarks, I'm not sure if there are known instances of people doing bad stuff with trademarks, but like you said having it explicitly stated probably does more good than bad. I see the trademark statement as the "other side of the coin" to the usual request OSS projects make of "if you use our project in your research please reference the project saying that you did", where this is "if you mention our project, don't misrepresent it".

Bottom line, these documents were all included in the Minimum Viable Government project and I don't see a reason not to include them other than it is extra stuff to read.

@rougier
Copy link
Contributor

rougier commented Aug 30, 2022

And now the stupid question: how do I approve? I don't see any button.

@kmuehlbauer
Copy link
Contributor

@rougier You would need to start a review by clicking on files on the top right of the PR I think.

@djhoese
Copy link
Member Author

djhoese commented Aug 31, 2022

@almarklein I know you are on paternity leave so no rush on reading through all this, but I think you're the last approval and then this can be merged.

@djhoese djhoese merged commit c398160 into vispy:main Sep 6, 2022
@djhoese djhoese deleted the doc-governance branch September 6, 2022 01:10
@djhoese
Copy link
Member Author

djhoese commented Sep 6, 2022

Thanks everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants