Skip to content

feat(sync): create spans to accurately record token usage#1

Open
GoldenChrysus wants to merge 1 commit into
comet-ml:mainfrom
GoldenChrysus:main
Open

feat(sync): create spans to accurately record token usage#1
GoldenChrysus wants to merge 1 commit into
comet-ml:mainfrom
GoldenChrysus:main

Conversation

@GoldenChrysus

Copy link
Copy Markdown

Found this tool today and excited to use it, but found that it doesn't do the one thing I wanted to use it for: observing my Bedrock costs in Opik.

Current implementation only creates traces instead of spans, so I've added functionality to create a span per assistant message.

I also fixed an issue where the tool currently doesn't respect the Opik project name at all. Claude Code is hard-coded as the value no matter what is present in the config file or env variables.

Sorry for all the white space changes. LSP does not like all the extra white space...

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