Disclaimer: Experimental PR review

Greptile Summary

This PR addresses how the langfuse Python SDK handles NaN (Not a Number) values during JSON serialization. The change modifies the EventSerializer class to serialize NaN values as the string "NaN" instead of None. This is a small but important change that improves consistency in how numerical edge cases are handled, particularly aligning with JavaScript's JSON handling and matching the existing behavior for infinity values in the codebase.

The change is implemented by adding a specific check for NaN float values using math.isnan() and returning the string "NaN" instead of the previous implicit None conversion. This modification helps maintain semantic clarity since None and NaN have distinct meanings in programming contexts.

Confidence score: 4/5

1. This PR is safe to merge as it's a focused change improving data serialization consistency
2. High confidence due to the small scope, clear purpose, and alignment with established serialization patterns
3. Files needing attention:
   • langfuse/_utils/serializer.py - verify this doesn't break any existing consumers expecting None values

_{1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile}