-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Assorted copy & URL updates #14222
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
Assorted copy & URL updates #14222
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 17.44kB (0.08%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
@@ -64,4 +64,6 @@ When your AI agents are part of larger applications (like web servers or APIs), | |||
|
|||
<Alert title="Agent Pipelines vs Direct LLM Calls"> | |||
|
|||
Creating an AI pipeline is different than calling an LLM. If you're creating AI pipelines by calling LLMs directly (without using a tool like LangChain), consider using [manual AI instrumentation](/platforms/python/tracing/instrumentation/custom-instrumentation/ai-agents-module). |
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.
@jaffrepaul - can you share why you put this alert at the bottom of the page? I would have assumed this information would be more useful closer to the beginning of the doc where we start with "once you've set up the SDK".
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.
The Alert component was already there, it just had no content in it 😀 There may be a larger convo around the best flow for this info, but (literally) filling in the gaps for now.
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.
This got removed in a different PR after chatting with the team ✅
5ae6f0c
to
b0df9da
Compare
Co-authored-by: Alex Krawiec <[email protected]>
Uh oh!
There was an error while loading. Please reload this page.