fix(api-models): usage unit to optional

[hassiebp]

Important

Make unit field optional in Model class in model.py.

• Models:
  • In model.py, changed unit field in Model class to be optional with a default of None.

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

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

Makes the unit field optional in the Model class, allowing model definitions without a specified usage unit in the Langfuse Python SDK.

• Modified langfuse/api/resources/commons/types/model.py to make unit field optional with typing.Optional[ModelUsageUnit]
• Potential breaking change for code that assumes unit is always present
• Aligns with CreateModelRequest where unit was already optional
• Impacts model usage tracking and cost calculations where unit type is required

_{1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

logic: Making unit optional could cause issues if any code relies on unit being present for cost calculations. Consider adding validation to ensure unit is present when input_price, output_price, or total_price are set.