Skip to content

Conversation

@Deepak-Kesavan
Copy link
Contributor

What

Send provider in the usage callback

Why

It is required for cost calculation

How

Get the provider from adapter class and set it in usage_kwargs

Relevant Docs

Related Issues or PRs

Dependencies Versions / Env Variables

Notes on Testing

...

Screenshots

...

Checklist

I have read and understood the Contribution Guidelines.

Signed-off-by: Deepak <89829542+Deepak-Kesavan@users.noreply.github.com>
Copy link
Contributor

@gaya3-zipstack gaya3-zipstack left a comment

Choose a reason for hiding this comment

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

Looks fine. Aapters 0.22.0 needs to be published for pdm.lock update

Copy link
Contributor

@chandrasekharan-zipstack chandrasekharan-zipstack left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Deepak <89829542+Deepak-Kesavan@users.noreply.github.com>
@Deepak-Kesavan Deepak-Kesavan merged commit c8d93f8 into main Jul 19, 2024
@Deepak-Kesavan Deepak-Kesavan deleted the feature/send-provider-in-the-usage-callback branch July 19, 2024 07:03
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.

4 participants