Skip to content

Reorganize repository structure into three main sections - #47

Open
CFLJacquet wants to merge 3 commits into
mainfrom
cfljacquet/reorg
Open

Reorganize repository structure into three main sections#47
CFLJacquet wants to merge 3 commits into
mainfrom
cfljacquet/reorg

Conversation

@CFLJacquet

Copy link
Copy Markdown
Collaborator
  • 1-tracing: Basic tracing examples and custom evaluations
  • 2-online-evaluation: Online evaluation examples and Ragas integration
  • 3-experiments: Experiments, eval integrations, notebooks, and scripts

- 1-tracing: Basic tracing examples and custom evaluations
- 2-online-evaluation: Online evaluation examples and Ragas integration
- 3-experiments: Experiments, eval integrations, notebooks, and scripts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@CFLJacquet
CFLJacquet requested a review from cdfox March 13, 2026 19:58
Comment thread 1-tracing/README.md Outdated

**[This notebook](./1-llm-span.ipynb)** shows you how to create and trace a simple LLM call.

<img src="./images/llm-span.png" height="350" >

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

When I preview this README (https://github.com/DataDog/llm-observability/blob/07baa86d22d5ea19ece3f840ad23342de9aff0f8/1-tracing/README.md) the image links are broken. I think it needs to be ../images/llm-span.png

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@CFLJacquet
CFLJacquet requested a review from cdfox March 13, 2026 22:11
Comment thread README.md
Comment on lines +28 to +29
git clone https://github.com/DataDog/llm-observability-cookbooks.git
cd llm-observability-cookbooks

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
git clone https://github.com/DataDog/llm-observability-cookbooks.git
cd llm-observability-cookbooks
git clone https://github.com/DataDog/llm-observability.git
cd llm-observability

@cdfox

cdfox commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Ok, I just started from scratch, starting from the README of this repo on your branch.

  • we don't need .env file in each directory, dotenv will find .env in the parent directory so user can just create once at top level of the repo
  • I think we should direct the user to start notebook from top level of repo, and then they can navigate into the subdirectories to run through the notebooks
  • we shouldn't need the READMEs within each subdirectory. I think it should just be the one README at top level of repo. Each notebook should stand contain any explanation needed about setup or how files are laid out. Notebooks are already a nice visual medium, once you have jupyter running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants