Skip to content

Conversation

poweiw
Copy link
Collaborator

@poweiw poweiw commented Jun 4, 2025

Adds an automatic labelling bot using LLM whenever an issue is raised. See an example issue at poweiw#4 and its run log at https://github.com/poweiw/TensorRT/actions/runs/15433437593

The goggles_action is still private. Please contact @poweiw for access.

This bot uses label descriptions defined in TRT repo, so ensure the labels descriptions are clear and instructive. Note that this bot DOES NOT auto-assign, so module owners still have to check their modules.

@poweiw poweiw marked this pull request as ready for review June 4, 2025 04:23
@poweiw poweiw requested a review from a team as a code owner June 4, 2025 04:23
@poweiw poweiw requested review from LeoZDong and akhilg-nv June 4, 2025 04:23
@akhilg-nv
Copy link
Collaborator

How is the LLM being queried? It seems to me like the labels on your sample issue don't necessarily match the existing pool of labels we have in the main repo, and I think we'd like it to do that for consistency and perhaps later auto-assigning based on the labels added.

@poweiw
Copy link
Collaborator Author

poweiw commented Jun 4, 2025

@akhilg-nv The labels are queried by the time the action was triggered. The labels are inconsistent to the labels we have in the main repo because the issue was raised in my fork. See the difference between labels in upstream and labels in my fork. Sent you the invitation to checkout the labeling logic

Copy link
Collaborator

@akhilg-nv akhilg-nv left a comment

Choose a reason for hiding this comment

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

@akhilg-nv The labels are queried by the time the action was triggered. The labels are inconsistent to the labels we have in the main repo because the issue was raised in my fork. See the difference between labels in upstream and labels in my fork. Sent you the invitation to checkout the labeling logic

Thanks, can we add a failsafe to make sure triaged label isn't applied incorrectly in the validation step? LGTM

@poweiw poweiw marked this pull request as draft June 14, 2025 00:23
@poweiw
Copy link
Collaborator Author

poweiw commented Jun 25, 2025

Thanks, can we add a failsafe to make sure triaged label isn't applied incorrectly in the validation step? LGTM

Fixed in e74434b

@poweiw poweiw force-pushed the dev-poweiw-trtllm-5596 branch from e74434b to 2b85d2b Compare June 25, 2025 22:28
@poweiw poweiw changed the base branch from release/10.11 to release/10.12 June 25, 2025 22:28
@poweiw poweiw marked this pull request as ready for review June 26, 2025 20:06
@poweiw poweiw force-pushed the dev-poweiw-trtllm-5596 branch from 2b85d2b to 0261f44 Compare June 26, 2025 20:09
poweiw added 5 commits June 26, 2025 13:09
Signed-off-by: Po-Wei Wang (Vincent) <[email protected]>
Signed-off-by: Po-Wei Wang (Vincent) <[email protected]>
Signed-off-by: Po-Wei Wang (Vincent) <[email protected]>
Signed-off-by: Po-Wei Wang (Vincent) <[email protected]>
Signed-off-by: Po-Wei Wang (Vincent) <[email protected]>
@poweiw poweiw force-pushed the dev-poweiw-trtllm-5596 branch from 0261f44 to 903ae62 Compare June 26, 2025 20:09
@poweiw poweiw merged commit bdca83e into NVIDIA:release/10.12 Jun 26, 2025
1 check passed
@poweiw poweiw deleted the dev-poweiw-trtllm-5596 branch July 14, 2025 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants