-
-
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
base: master
Are you sure you want to change the base?
Assorted copy & URL updates #14222
Conversation
Co-authored-by: paul.jaffre <[email protected]>
Co-authored-by: paul.jaffre <[email protected]>
Co-authored-by: paul.jaffre <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 862.62kB (4.09%) ⬆️. 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.
Co-authored-by: Shannon Anahata <[email protected]>
Uh oh!
There was an error while loading. Please reload this page.