fix: response model would overwrite request model

Behavior: Fix get_llm_usage_for_span in utils.rs to correctly use response_model instead of request_model for response calculations.
Misc: Fix typo in comment in attributes.rs .

lmnr-ai / lmnr

Laminar - open-source all-in-one platform for engineering AI products. Crate data flywheel for you AI app. Traces, Evals, Datasets, Labels. YC S24.

https://www.lmnr.ai

Apache License 2.0

1.17k stars 58 forks source link

fix: response model would overwrite request model #238

Closed dinmukhamedm closed 3 days ago

dinmukhamedm commented 3 days ago

[!IMPORTANT] Fix incorrect model usage in get_llm_usage_for_span in utils.rs and correct a typo in attributes.rs.

Behavior:

Fix get_llm_usage_for_span in utils.rs to correctly use response_model instead of request_model for response calculations.

Misc:

Fix typo in comment in attributes.rs.

^{This description was created by}^{for b6e8384c32579f2f97e135026eb16f2dc94f44cb. It will automatically update as commits are pushed.}