Skip to content
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

[MLOB-2058] Add Node.js Quickstart Sections for LLM Observability #27211

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

sabrenner
Copy link
Contributor

What does this PR do? What is the motivation?

Adds Node.js code blocks for the LLM Observability Quickstart guide.

Merge instructions

Merge readiness:

  • Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Comment on lines 105 to 109
```shell
DD_LLMOBS_ENABLED=1 DD_LLMOBS_ML_APP=onboarding-quickstart \
DD_API_KEY=<YOUR_DATADOG_API_KEY> DD_SITE={{< region-param key="dd_site" >}} \
DD_LLMOBS_AGENTLESS_ENABLED=1 ddtrace-run python quickstart.py
```
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this now seems to render the DD_SITE portion as:

DD_SITE=<span class="js-region-param region-param" data-region-param="dd_site"></span>

without any changes to the actual block, similar for the Node.js tab. Do these region shortcodes not work inside of tab shortcodes?


3. Run the script with the following shell command. This sends a trace of the OpenAI call to Datadog.

{{< tabs >}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

adding tabs here seems to make the page jump when clicking between tabs, but just for this section.

@sabrenner sabrenner marked this pull request as ready for review January 21, 2025 20:14
@sabrenner sabrenner requested a review from a team as a code owner January 21, 2025 20:14
@cswatt
Copy link
Contributor

cswatt commented Jan 21, 2025

Added DOCS-9911 to review

@cswatt cswatt added the editorial review Waiting on a more in-depth review label Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial review Waiting on a more in-depth review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants