Thanks to visit codestin.com
Credit goes to github.com

Skip to content

LLM Observability Datasets and Experiments GA Documentation #31619

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Sep 16, 2025
Merged

LLM Observability Datasets and Experiments GA Documentation #31619

merged 10 commits into from
Sep 16, 2025

Conversation

alexbarksdale
Copy link
Member

What does this PR do? What is the motivation?

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:

Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.

Additional notes

@github-actions GitHub Actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@CFLJacquet CFLJacquet marked this pull request as ready for review September 15, 2025 20:10
@CFLJacquet CFLJacquet requested a review from a team as a code owner September 15, 2025 20:10
@drichards-87
Copy link
Contributor

Created DOCS-12075 for Docs Team review.

@drichards-87 drichards-87 added the editorial review Waiting on a more in-depth review label Sep 15, 2025
@alexbarksdale alexbarksdale added the Do Not Merge Just do not merge this PR :) label Sep 16, 2025
@cswatt
Copy link
Contributor

cswatt commented Sep 16, 2025

I pushed mostly grammar/syntax edits, though I did make one structural change:

Previously, the "Working with CSV" section included both how to create a dataset from a CSV, and how to export a dataset to a pandas DataFrame. It seems to me that exporting to pandas doesn't necessarily have to do with importing CSVs, so I moved the as_dataframe() content into a new "Exporting a dataset to pandas" section.

This means:

  • The instructions to create a dataset from a CSV no longer tell you to pip install pandas
  • The ability to export a dataset to pandas is no longer tied to the dataset having been created from a CSV

Can we confirm that the above are okay/accurate?

@gary-huang
Copy link
Contributor

I pushed mostly grammar/syntax edits, though I did make one structural change:

Previously, the "Working with CSV" section included both how to create a dataset from a CSV, and how to export a dataset to a pandas DataFrame. It seems to me that exporting to pandas doesn't necessarily have to do with importing CSVs, so I moved the as_dataframe() content into a new "Exporting a dataset to pandas" section.

This means:

  • The instructions to create a dataset from a CSV no longer tell you to pip install pandas
  • The ability to export a dataset to pandas is no longer tied to the dataset having been created from a CSV

Can we confirm that the above are okay/accurate?

Great catch @cswatt ! You are correct, pandas is only needed for as_dataframe, creating from CSV does not need pandas. Thank you!

@cswatt cswatt removed the Do Not Merge Just do not merge this PR :) label Sep 16, 2025
@cswatt cswatt merged commit 17cb8e0 into master Sep 16, 2025
19 of 22 checks passed
@cswatt cswatt deleted the alex.barksdale/update-llm-experiments branch September 16, 2025 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cswatt cswatt cswatt approved these changes

Assignees
No one assigned
Labels
editorial review Waiting on a more in-depth review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@alexbarksdale @drichards-87 @cswatt @gary-huang @CFLJacquet