fix(serializer): NaN to string rather than None

[hassiebp]

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}

---

Important

Change serialization of NaN values from None to "NaN" in default() method of EventSerializer.

• Behavior:
  • Change serialization of NaN values from None to "NaN" in default() method of EventSerializer in serializer.py.

^{This description was created by for 4c533e0. You can customize this summary. It will automatically update as commits are pushed.}

[greptile-apps]

Reviewing changes made in this pull request