|
1 | 1 | # How to become a core contributor
|
2 | 2 |
|
3 |
| -Thanks for your interest in joining the Lightning team! We’re a rapidly growing project which is poised to become the go-to framework for DL researchers! |
4 |
| -We're currently recruiting for a team of 5 core maintainers. |
| 3 | +Thanks for your interest in joining the Lightning team! We’re a rapidly growing project which is poised to become the go-to framework for DL researchers! |
| 4 | +We're currently recruiting for a team of 5 core maintainers. |
5 | 5 |
|
6 | 6 | As a core maintainer you will have a strong say in the direction of the project. Big changes will require a majority of maintainers to agree.
|
7 | 7 |
|
8 |
| -### Code of conduct |
| 8 | +### Code of conduct |
9 | 9 | First and foremost, you'll be evaluated against [these core values](https://github.com/PyTorchLightning/pytorch-lightning/blob/master/.github/CONTRIBUTING.md). Any code we commit or feature we add needs to align with those core values.
|
10 | 10 |
|
11 |
| -### The bar for joining the team |
| 11 | +### The bar for joining the team |
12 | 12 | Lightning is being used to solve really hard problems at the top AI labs in the world. As such, the bar for adding team members is extremely high. Candidates must have solid engineering skills, have a good eye for user experience, and must be a power user of Lightning and PyTorch.
|
13 | 13 |
|
14 | 14 | With that said, the Lightning team will be diverse and a reflection of an inclusive AI community. You don't have to be an engineer to contribute! Scientists with great usability intuition and PyTorch ninja skills are welcomed!
|
@@ -36,26 +36,26 @@ Pleasant/helpful tone.
|
36 | 36 | - Code is NOT overly engineered or hard to read
|
37 | 37 | - Ask yourself, could a non-engineer understand what’s happening here?
|
38 | 38 | - Make sure new tests are written
|
39 |
| -- Is this NECESSARY for Lightning? There are some PRs which are just purely about adding engineering complexity which have no place in Lightning. |
| 39 | +- Is this NECESSARY for Lightning? There are some PRs which are just purely about adding engineering complexity which have no place in Lightning. |
40 | 40 | Guidance
|
41 | 41 | - Some other PRs are for people who are wanting to get involved and add something unnecessary. We do want their help though! So don’t approve the PR, but direct them to a Github issue that they might be interested in helping with instead!
|
42 |
| -- To be considered for core contributor, please review 10 PRs and help the authors land it on master. Once you've finished the review, ping me |
| 42 | +- To be considered for core contributor, please review 10 PRs and help the authors land it on master. Once you've finished the review, ping me |
43 | 43 | for a sanity check. At the end of 10 PRs if your PR reviews are inline with expectations described above, then you can merge PRs on your own going forward,
|
44 | 44 | otherwise we'll do a few more until we're both comfortable :)
|
45 | 45 |
|
46 | 46 | #### Project directions
|
47 | 47 | There are some big decisions which the project must make. For these I expect core contributors to have something meaningful to add if it’s their area of expertise.
|
48 | 48 |
|
49 | 49 | #### Diversity
|
50 |
| -Lightning should reflect the broader community it serves. As such we should have scientists/researchers from |
51 |
| -different fields contributing! |
| 50 | +Lightning should reflect the broader community it serves. As such we should have scientists/researchers from |
| 51 | +different fields contributing! |
52 | 52 |
|
53 | 53 | The first 5 core contributors will fit this profile. Thus if you overlap strongly with experiences and expertise as someone else on the team, you might have to wait until the next set of contributors are added.
|
54 | 54 |
|
55 | 55 | #### Summary: Requirements to apply
|
56 | 56 | The goal is to be inline with expectations for solving issues by the last one so you can do them on your own. If not, I might ask you to solve a few more specific ones.
|
57 | 57 |
|
58 |
| -- Solve 10+ Github issues. |
| 58 | +- Solve 10+ Github issues. |
59 | 59 | - Create 5+ meaningful PRs which solves some reported issue - bug,
|
60 | 60 | - Perform 10+ PR reviews from other contributors.
|
61 | 61 |
|
|
0 commit comments