-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
feat(llm-detector): New group category for LLM Detected Issues #103089
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| type_id = 2011 | ||
| slug = "profile_function_regression" | ||
| description = "Function Regression" | ||
| category = GroupCategory.PERFORMANCE.value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this one change as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think so - i had just picked METRIC the first time around as a placeholder. i think for this issue type, metric is the correct category
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah whoops the diff got me, i meant for this comment to be about line 627. are non v2 categories still used at this point?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it - i don't think so, issue-taxonomy flag is at 100 GA i believe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may be worth removing the old ones at some point 👍🏼 thanks for checking!
src/sentry/issues/grouptype.py
Outdated
| FRONTEND = 14 | ||
| MOBILE = 15 | ||
|
|
||
| LLM_DETECTED = 16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: maybe AI_DETECTED or AI_SUGGESTED? (I suppose we can change however this is displayed at any rate)
LLM detected issues will be from a variety of different categories so there's no easy way to categorize them with the existing categories- making a new category to support them
fixes https://linear.app/getsentry/issue/ID-1083/add-new-issue-category-for-llm-detected-issues