Skip to content

Commit 28315f8

Browse files
Add approvers & include SME docs
1 parent 55fea6d commit 28315f8

File tree

3 files changed

+78
-4
lines changed

3 files changed

+78
-4
lines changed

.github/pull_request_template.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Self checklist (all need to be checked):
2828
If you have UI changes:
2929
<!--- You can ignore these if you are doing manifest, backend, internal logic, etc changes; aka non-UI / visual changes -->
3030
- [ ] Included any necessary screenshots or gifs if it was a UI change.
31-
- [ ] Included tags to the UX team if it was a UI/UX change.
31+
- [ ] 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).
3232

3333
After the PR is posted & before it merges:
3434
- [ ] The developer has tested their solution on a cluster by using the image produced by the PR to `main`

OWNERS

+12-3
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,17 @@
11
approvers:
22
- andrewballantyne
3+
- lucferbux
4+
- alexcreasy
5+
- christianvogt
36

47
reviewers:
5-
- DaoDaoNoCode
6-
- lucferbux
7-
- Gkrumbach07
88
- alexcreasy
9+
- christianvogt
10+
- uidoyen
11+
- Gkrumbach07
12+
- lucferbux
13+
- DaoDaoNoCode
14+
- manaswinidas
15+
- pnaik1
16+
- ppadti
17+
- dpanshug

docs/smes.md

+65
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
# Subject Matter Experts (SMEs)
2+
3+
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.
4+
5+
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.
6+
7+
Below there will be some terms like “previous” and “backup”, these are for additional context. The way you can read each are as follows:
8+
- **previous** – the initial SME in the area. If you need legacy context, this person may be able to help
9+
- **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
10+
- **and** – Ping both during conversations – could be onboarding, could be a need to share information, best get both people involved at the same time
11+
12+
General Dashboard ownership
13+
- Infrastructure / direction
14+
- Architect: `Andrew` ([andrewballantyne])
15+
- General UX: `Kyle` ([kywalker-rh])
16+
- App Text: `Katie` ([kaedward])
17+
- Testing (Integration, Unit, etc)
18+
- Area lead: `Gage` ([Gkrumbach07])
19+
- Performance
20+
- Area lead: `Lucas` ([lucferbux])
21+
22+
Dashboard features
23+
- Data Science Projects
24+
- Feature lead: `Andrew` ([andrewballantyne])
25+
- UX: `Kyle` ([kywalker-rh]) **and** `Kun` ([xianli123])
26+
- Data Science Pipelines
27+
- Feature lead: `Andrew` ([andrewballantyne])
28+
- UX: `Yan` ([yannnz])
29+
- Previous: `Kyle` ([kywalker-rh])
30+
- Explainability, Bias
31+
- Feature lead: `Alex` ([alexcreasy])
32+
- UX: `Vince` ([vconzola])
33+
- Model Serving (Custom runtimes, general Model Serving)
34+
- Feature lead: `Lucas` ([lucferbux])
35+
- UX: `Vince` ([vconzola])
36+
- Model Serving - Performance metrics
37+
- Feature lead: `Andrew` ([andrewballantyne])
38+
- UX: `Vince` ([vconzola])
39+
- BYON - Custom Notebook Images
40+
- Feature lead: `Juntao` ([DaoDaoNoCode])
41+
- Backup: `Andrew` ([andrewballantyne])
42+
- UX: `Vince` ([vconzola])
43+
- Habana / Accelerators
44+
- Feature lead: `Gage` ([Gkrumbach07]);
45+
- Backup: `Andrew` ([andrewballantyne])
46+
- UX: `Yan` ([yannnz])
47+
- Model Registry
48+
- Feature lead: TBD
49+
- UX: `Sim` ([simrandhaliw]) **and** `Haley` ([yih-wang])
50+
51+
<!-- Developers -->
52+
[andrewballantyne]: https://github.com/andrewballantyne
53+
[Gkrumbach07]: https://github.com/Gkrumbach07
54+
[lucferbux]: https://github.com/lucferbux
55+
[alexcreasy]: https://github.com/alexcreasy
56+
[DaoDaoNoCode]: https://github.com/DaoDaoNoCode
57+
58+
<!-- UX -->
59+
[kywalker-rh]: https://github.com/kywalker-rh
60+
[kaedward]: https://github.com/kaedward
61+
[xianli123]: https://github.com/xianli123
62+
[vconzola]: https://github.com/vconzola
63+
[yannnz]: https://github.com/yannnz
64+
[simrandhaliw]: https://github.com/simrandhaliw
65+
[yih-wang]: https://github.com/yih-wang

0 commit comments

Comments
 (0)