-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
feat(insights): move insights to domain focused categorization #11673
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
580a6f1 to
032d175
Compare
|
We shouldn't merge this until @DominikB2014's work is close to ready for GA. |
Bundle ReportChanges will increase total bundle size by 1.75kB (0.01%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
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.
Lgtm!
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.
All changes and redirects look good!
19aa0b9 to
ad6685a
Compare
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.
Thanks again for putting up this huge PR! Couple of notes:
-I think it would be good to combine the AI Performance + LLM Monitoring page (the same way you did for mobile). Otherwise, there's just not really enough content on the standalone page
-It would also be good to add screenshots of the overview pages (I can help with this.)
-I added a little bit more meat to the frontend/backend pages. If there's anything else you can think of to add, it would be good to make those pages a bit more meaty too
docs/product/insights/ai/index.mdx
Outdated
| --- | ||
| title: "AI Performance" | ||
| sidebar_order: 135 | ||
| description: "Performance views for AI developers." |
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.
| description: "Performance views for AI developers." | |
| description: "Learn how to use Sentry's AI Performance tool to monitor LLM calls." |
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.
Large Language Models are just one type of AI workflow that we happen to have concentrated on first. Another example where I could imagine us having some monitoring is generative image models (as an example).
Even though the AI page is a bit sparse, my preference would be to not use monitoring LLM calls as a synonym for AI.
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.
That makes sense. Let me take another pass at adding a bit more context here...
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.
How about: Learn how to use Sentry's AI Performance tool to get insights into things that may be affecting your application health, including critical LLM metrics.
We can make updates here as we add new AI capabilities. Feel free to update if you have a broader way to speak to this tools without specifically mentioning LLM.
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
28b30d8 to
2b266dd
Compare
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.
Thanks so much for adding screenshots! I've made a few more tweaks and added a bit more content to the AI Perf page. It makes sense that you want to leave it more broad and not just have it be about LLMs. I think we're in good shape!
docs/product/insights/ai/index.mdx
Outdated
| --- | ||
| title: "AI Performance" | ||
| sidebar_order: 135 | ||
| description: "Performance views for AI developers." |
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.
How about: Learn how to use Sentry's AI Performance tool to get insights into things that may be affecting your application health, including critical LLM metrics.
We can make updates here as we add new AI capabilities. Feel free to update if you have a broader way to speak to this tools without specifically mentioning LLM.
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
Co-authored-by: Liza Mock <[email protected]>
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.
Just a few tiny tweaks, but it's looking good!
|
|
||
| <Include name="feature-limited-on-team-retention.mdx" /> | ||
|
|
||
| Sentry's [**Frontend Performance**](https://sentry.io/orgredirect/organizations/:orgslug/insights/backend/) page gives you an overview of the health of your application. You'll be able to see things like **Best Page Opportunities** (the improvements that would most help increase your performance score), your **Most Time-Consuming Assets**, **p50** and **p75 Duration**, and so on. |
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.
| Sentry's [**Frontend Performance**](https://sentry.io/orgredirect/organizations/:orgslug/insights/backend/) page gives you an overview of the health of your application. You'll be able to see things like **Best Page Opportunities** (the improvements that would most help increase your performance score), your **Most Time-Consuming Assets**, **p50** and **p75 Duration**, and so on. | |
| Sentry's [**Frontend Performance**](https://sentry.io/orgredirect/organizations/:orgslug/insights/frontend/) page gives you an overview of the health of your application. You'll be able to see things like **Best Page Opportunities** (the improvements that would most help increase your performance score), your **Most Time-Consuming Assets**, **p50** and **p75 Duration**, and so on. |
| <Include name="feature-limited-on-team-retention.mdx" /> | ||
|
|
||
| <Note> | ||
| Sentry’s frontend, backend, mobile, and AI performance insights (all under the [**Insights**](https://sentry.io/orgredirect/organizations/:orgslug/insights/) heading in Sentry), give you a clear summary of the telemetry and events impacting the performance of individual parts of your application. Each section shows a high-level overview of your application's health with widgets for "Best Page Opportunities" (improvement that will be most helpful for boosting your performance score), most time-consuming events, and so on. You can also drill deeper into specific workflows to see specific traces, errors, and releases. This helps you identify potential problems and spot trends, fix bugs, and ultimately boost application performance. |
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.
| Sentry’s frontend, backend, mobile, and AI performance insights (all under the [**Insights**](https://sentry.io/orgredirect/organizations/:orgslug/insights/) heading in Sentry), give you a clear summary of the telemetry and events impacting the performance of individual parts of your application. Each section shows a high-level overview of your application's health with widgets for "Best Page Opportunities" (improvement that will be most helpful for boosting your performance score), most time-consuming events, and so on. You can also drill deeper into specific workflows to see specific traces, errors, and releases. This helps you identify potential problems and spot trends, fix bugs, and ultimately boost application performance. | |
| Sentry’s frontend, backend, mobile, and AI performance insights (all under the [**Insights**](https://sentry.io/orgredirect/organizations/:orgslug/insights/) heading in Sentry), give you a clear summary of the telemetry and events impacting the performance of individual parts of your application. Each section shows a high-level overview of your application's health with widgets for **Best Page Opportunities** (improvement that will be most helpful for boosting your performance score), most time-consuming events, and so on. You can also drill deeper into specific workflows to see specific traces, errors, and releases. This helps you identify potential problems and spot trends, fix bugs, and ultimately boost application performance. |
Updating this so it's consistent with how we have it on other pages.
|
|
||
| Sentry's [**AI Performance**](https://sentry.io/orgredirect/organizations/:orgslug/insights/ai/) page gives you an overview of the health of your application. You'll be able to see things like **Transactions Per Minute**, **p50** and **p75 Duration**, and so on. | ||
|
|
||
| You can also dive deeper into LLM Monitoring and get insights into critical LLM metrics, like token usage, to monitor and fix issues with AI pipelines. In addition to having a dedicated space to monitor AI performance, you can also look at Sentry's Insights tab to monitor backend, mobile, and frontend performance. |
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.
| You can also dive deeper into LLM Monitoring and get insights into critical LLM metrics, like token usage, to monitor and fix issues with AI pipelines. In addition to having a dedicated space to monitor AI performance, you can also look at Sentry's Insights tab to monitor backend, mobile, and frontend performance. | |
| You can also dive deeper into LLM Monitoring and get insights into critical LLM metrics, like token usage, to monitor and fix issues with AI pipelines. In addition to having a dedicated space to monitor AI performance, you can also look at Sentry's **Insights** tab to monitor backend, mobile, and frontend performance. |
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.
Everything looks good and works correctly, added a question about one of the page descriptions, besides that, good to go!
DESCRIBE YOUR PR
Move insights to domain focused hierarchy from blob of views.
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
LEGAL BOILERPLATE
Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
EXTRA RESOURCES