[Misc] Add CODEOWNERS to the project. #988
Conversation
DescriptionStart with a short description of what the PR does and how this is a change from The rest of the description includes relevant details and context, examples:
If the change fixes a bug or a Github issue, please include a link, e.g.,: TestsPlease describe how you tested this change, and include any instructions and/or ChecklistBefore submitting this PR, please make sure:
|
py4
force-pushed
the
pooyam
branch
2 times, most recently
from
749fd74 to
00265e5
Compare
Signed-off-by: Pooya Moradi <[email protected]>
|
Awesome. If I understand correctly, it will now require an approval from one of the owners before it can be merged right? |
vipannalla
left a comment
vipannalla
left a comment
There was a problem hiding this comment.
Thanks for the PR, I suggested few owner changes but LGTM lets merge and iterate forward.
| /.buildkite/ @jcyang43 | ||
| /.github/ @jcyang43 |
| /tpu_inference/spec_decode/ @py4 @Lumosis | ||
|
|
||
| # Platform Support | ||
| /tpu_inference/platforms/ @xiangxu-google @lsy323 |
There was a problem hiding this comment.
Since Siyuan is leaving and Xiang is OOO for few months, can you probably add @wenxindongwork instead who is updating this for DP support.
| /tpu_inference/runner/persistent_batch_manager.py @py4 @xiangxu-google | ||
| /tpu_inference/runner/speculative_decoding_manager.py @py4 @Lumosis | ||
| /tpu_inference/executors/ @xiangxu-google @mrjunwan-lang | ||
| /tpu_inference/core/ @sixiang-google |
|
|
||
| # JAX Model Layers - Attention | ||
| /tpu_inference/layers/jax/ @py4 @bzgoogle @jrplatin @gpolovets1 | ||
| /tpu_inference/layers/vllm/ @hfan @vanbasten23 @lsy323 |
There was a problem hiding this comment.
Similar to comment below, can you add me instead of Siyuan for various vLLM related things?
No description provided.