Skip to content

Commit

Permalink
Add approvers & include SME docs
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewballantyne committed Aug 29, 2023
1 parent 55fea6d commit 28315f8
Show file tree
Hide file tree
Showing 3 changed files with 78 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ Self checklist (all need to be checked):
If you have UI changes:
<!--- You can ignore these if you are doing manifest, backend, internal logic, etc changes; aka non-UI / visual changes -->
- [ ] Included any necessary screenshots or gifs if it was a UI change.
- [ ] Included tags to the UX team if it was a UI/UX change.
- [ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the [SMEs](https://github.com/opendatahub-io/odh-dashboard/tree/main/docs/smes.md) section).

After the PR is posted & before it merges:
- [ ] The developer has tested their solution on a cluster by using the image produced by the PR to `main`
15 changes: 12 additions & 3 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -1,8 +1,17 @@
approvers:
- andrewballantyne
- lucferbux
- alexcreasy
- christianvogt

reviewers:
- DaoDaoNoCode
- lucferbux
- Gkrumbach07
- alexcreasy
- christianvogt
- uidoyen
- Gkrumbach07
- lucferbux
- DaoDaoNoCode
- manaswinidas
- pnaik1
- ppadti
- dpanshug
65 changes: 65 additions & 0 deletions docs/smes.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
# Subject Matter Experts (SMEs)

A given subject matter expert is not necessarily the most knowledgeable in the area, but they are the one who probably knows the most about it when it was originally done or has a responsibility to expand their knowledge to know about the area going forward. Contacting them first will help with delegation of responsibilities at the Dashboard level.

This will detail out former (or current) feature leads, area leads (has a responsibility to understand the area), as well as any other notable position in relation to the area. If you need to talk to someone or ping someone for a review, this information should help you determine who.

Below there will be some terms like “previous” and “backup”, these are for additional context. The way you can read each are as follows:
- **previous** – the initial SME in the area. If you need legacy context, this person may be able to help
- **backup** – a good person to lean on if there is a need for any 2nd opinions, for bouncing ideas off of, or any larger discussion about direction
- **and** – Ping both during conversations – could be onboarding, could be a need to share information, best get both people involved at the same time

General Dashboard ownership
- Infrastructure / direction
- Architect: `Andrew` ([andrewballantyne])
- General UX: `Kyle` ([kywalker-rh])
- App Text: `Katie` ([kaedward])
- Testing (Integration, Unit, etc)
- Area lead: `Gage` ([Gkrumbach07])
- Performance
- Area lead: `Lucas` ([lucferbux])

Dashboard features
- Data Science Projects
- Feature lead: `Andrew` ([andrewballantyne])
- UX: `Kyle` ([kywalker-rh]) **and** `Kun` ([xianli123])
- Data Science Pipelines
- Feature lead: `Andrew` ([andrewballantyne])
- UX: `Yan` ([yannnz])
- Previous: `Kyle` ([kywalker-rh])
- Explainability, Bias
- Feature lead: `Alex` ([alexcreasy])
- UX: `Vince` ([vconzola])
- Model Serving (Custom runtimes, general Model Serving)
- Feature lead: `Lucas` ([lucferbux])
- UX: `Vince` ([vconzola])
- Model Serving - Performance metrics
- Feature lead: `Andrew` ([andrewballantyne])
- UX: `Vince` ([vconzola])
- BYON - Custom Notebook Images
- Feature lead: `Juntao` ([DaoDaoNoCode])
- Backup: `Andrew` ([andrewballantyne])
- UX: `Vince` ([vconzola])
- Habana / Accelerators
- Feature lead: `Gage` ([Gkrumbach07]);
- Backup: `Andrew` ([andrewballantyne])
- UX: `Yan` ([yannnz])
- Model Registry
- Feature lead: TBD
- UX: `Sim` ([simrandhaliw]) **and** `Haley` ([yih-wang])

<!-- Developers -->
[andrewballantyne]: https://github.com/andrewballantyne
[Gkrumbach07]: https://github.com/Gkrumbach07
[lucferbux]: https://github.com/lucferbux
[alexcreasy]: https://github.com/alexcreasy
[DaoDaoNoCode]: https://github.com/DaoDaoNoCode

<!-- UX -->
[kywalker-rh]: https://github.com/kywalker-rh
[kaedward]: https://github.com/kaedward
[xianli123]: https://github.com/xianli123
[vconzola]: https://github.com/vconzola
[yannnz]: https://github.com/yannnz
[simrandhaliw]: https://github.com/simrandhaliw
[yih-wang]: https://github.com/yih-wang

0 comments on commit 28315f8

Please sign in to comment.