Skip to content

Trace LLMP completions in Langfuse - #178

Open
mbrotos wants to merge 1 commit into
VectorInstitute:mainfrom
mbrotos:codex/llmp-langfuse-trace
Open

Trace LLMP completions in Langfuse#178
mbrotos wants to merge 1 commit into
VectorInstitute:mainfrom
mbrotos:codex/llmp-langfuse-trace

Conversation

@mbrotos

@mbrotos mbrotos commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add an explicit nested Langfuse Generation around LLMP completions
  • record request messages, response content, model, and token usage
  • add focused coverage and document the tracing behavior

Validation

  • focused LLMP client pytest suite
  • mypy on aieng
  • Ruff format and check

Co-authored-by: Codex <noreply@openai.com>
@mbrotos
mbrotos force-pushed the codex/llmp-langfuse-trace branch from 8623e15 to 7d4c047 Compare August 3, 2026 15:07
@mbrotos

mbrotos commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

CI note: the only failing check is nbqa-ruff on the pre-existing archived notebook playground/energy_case_study/notebooks/01_energy_oil_case_study.ipynb (PLR0917: six positional arguments). That file is outside this PR; uv-lock, Ruff, and mypy pass.

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.

1 participant