implement LLM Obs SDK spans APIs by gary-huang · Pull Request #8390 · DataDog/dd-trace-java · GitHub
Skip to content

implement LLM Obs SDK spans APIs#8390

Merged
gary-huang merged 17 commits into
gary/llmobs-sdk-mergefrom
gary/impl-llm-obs-api
Jun 4, 2025
Merged

implement LLM Obs SDK spans APIs#8390
gary-huang merged 17 commits into
gary/llmobs-sdk-mergefrom
gary/impl-llm-obs-api

Conversation

@gary-huang

@gary-huang gary-huang commented Feb 14, 2025

Copy link
Copy Markdown
Contributor

What Does This Do

adds the implementation for llm obs spans

see #8135 for list of APIs that is implemented and usage

Motivation

Additional Notes

to be merged to the branch gary/llmobs-sdk-merge after #8135 has been merged

Contributor Checklist

Jira ticket: MLOB-1462

@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch from 3bcbe81 to c437b9c Compare February 19, 2025 14:47
@pr-commenter

pr-commenter Bot commented Feb 19, 2025

Copy link
Copy Markdown

@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch 3 times, most recently from 871d358 to 180ccf5 Compare February 20, 2025 21:17
@github-actions

Copy link
Copy Markdown
Contributor

Hi! 👋 Looks like you updated a Git Submodule.
If this was not intentional please make sure to:

@gary-huang gary-huang changed the base branch from master to gary/add-llm-obs-api February 20, 2025 21:18
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch 5 times, most recently from 94136c5 to 7064f57 Compare February 20, 2025 23:13
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch 2 times, most recently from 9cb3e0a to 76a4b2f Compare March 5, 2025 19:00
@gary-huang gary-huang force-pushed the gary/add-llm-obs-api branch from 77ddec8 to cb95fbb Compare March 5, 2025 19:04
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch 5 times, most recently from a55367c to 40dcdfe Compare March 6, 2025 20:51
@gary-huang gary-huang force-pushed the gary/add-llm-obs-api branch from 7f41706 to a9ec5dd Compare March 6, 2025 21:05
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch from 40dcdfe to acde475 Compare March 6, 2025 21:07
@gary-huang gary-huang force-pushed the gary/add-llm-obs-api branch from a9ec5dd to c75df4b Compare March 6, 2025 23:01
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch from acde475 to 829ffb1 Compare March 7, 2025 00:00
@gary-huang gary-huang force-pushed the gary/add-llm-obs-api branch 2 times, most recently from c1f63dc to 6e250a2 Compare March 7, 2025 13:34
@gary-huang gary-huang force-pushed the gary/impl-llm-obs-api branch 4 times, most recently from d548421 to d9a57b3 Compare March 7, 2025 14:06
@gary-huang gary-huang requested review from a team as code owners May 14, 2025 18:18
@gary-huang gary-huang requested review from jandro996, jpbempel, smola and tonyredondo and removed request for a team May 14, 2025 18:18
@gary-huang gary-huang force-pushed the gary/llmobs-sdk-merge branch from 896f40a to 2be1b73 Compare May 14, 2025 18:23
@gary-huang gary-huang removed request for a team, jandro996, jpbempel, smola and tonyredondo May 14, 2025 18:23
@gary-huang

Copy link
Copy Markdown
Contributor Author

@gary-huang gary-huang merged commit 77291d8 into gary/llmobs-sdk-merge Jun 4, 2025
516 of 520 checks passed
@gary-huang gary-huang deleted the gary/impl-llm-obs-api branch June 4, 2025 19:35
gary-huang added a commit that referenced this pull request Jul 9, 2025
* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* spotless

* add APIs for llm obs sdk (#8135)

* add APIs for llm obs

* add llm message class to support llm spans

* follow java convention of naming Id instead of ID

* add codeowners

* rename ID to Id according to java naming conventions

* Undo change to integrations-core submodule

* fix build gradle

* rm empty line

* fix test
nayeem-kamal added a commit that referenced this pull request Jul 9, 2025
* add APIs for llm obs sdk (#8135)

* add APIs for llm obs

* add llm message class to support llm spans

* follow java convention of naming Id instead of ID

* add codeowners

* implement LLM Obs SDK spans APIs (#8390)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* spotless

* add APIs for llm obs sdk (#8135)

* add APIs for llm obs

* add llm message class to support llm spans

* follow java convention of naming Id instead of ID

* add codeowners

* rename ID to Id according to java naming conventions

* Undo change to integrations-core submodule

* fix build gradle

* rm empty line

* fix test

* LLM Obs SDK Mapper (#8372)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>

* updated to master submodule

* LLM Obs SDK use context API for parent children span linkage (#8711)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* impl llmobs agent and llmobs apis

* use new ctx api to track parent span

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* ran spotless

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>

* LLM Obs SDK evaluation metrics submission (#8688)

* add APIs for llm obs

* add llm message class to support llm spans

* add llm message class to support llm spans

* impl llmobs agent and llmobs apis

* support llm messages with tool calls

* handle default model name and provider

* rm unneeded file

* impl llmobs agent and llmobs apis

* impl llmobs agent

* working writer

* add support for llm message and tool calls

* impl llmobs agent and llmobs apis

* use new ctx api to track parent span

* add api for evals

* working impl supporting both agentless and agent

* handle null tags and default to default ml app if null or empty string provided in the override

* cleaned up whitespace

* resolve merge conflicts

* remaining merge conflicts

* fix bad method call

* fixed llmobs intake creation if llmobs not enabled

* removed print statements

* ran spotless

* ran spotless

* added tests for llmobsspanmapper

* fixed coverage for tags

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>

* fix CODEOWNERS

---------

Co-authored-by: Nayeem Kamal <nayeem.kamal@datadoghq.com>
Co-authored-by: Nayeem Kamal <kamal.nayeem12@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: mlobs ML Observability (LLMObs) tag: experimental Experimental changes type: enhancement Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants