Skip to content
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.

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

Allow AI to be nullable #74746

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Allow AI to be nullable #74746

merged 1 commit into from
Jul 23, 2024

Conversation

colin-sentry
Copy link
Member

AI is not currently nullable, it would be nice if it was.

This makes ai.total_tokens.used display properly in the frontend if ai.total_cost has never been sent as a metric

@colin-sentry colin-sentry requested review from a team as code owners July 23, 2024 17:52
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 23, 2024
@colin-sentry colin-sentry enabled auto-merge (squash) July 23, 2024 17:56
@colin-sentry colin-sentry merged commit cf6a758 into master Jul 23, 2024
50 checks passed
@colin-sentry colin-sentry deleted the nullable_ai branch July 23, 2024 18:23
@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants