fix: tracing of non-serializable values, e.g. bytes #34
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes traces when non-serializable data (like bytes from images, videos, audio, etc.) are present.
Before fixes (output incorrectly shows
<replaced>
): https://us.cloud.langfuse.com/project/cmaoqkqd901afad07ui6ndt1v/traces/36829363ba6e9f5177322c5edaf3b0fc?timestamp=2025-05-17T17:03:02.274Z&display=detailsAfter fixes (only the image_reader tool output bytes show
<replaced>
, which is correct): https://us.cloud.langfuse.com/project/cmaoqkqd901afad07ui6ndt1v/traces/0ee3d42598ab472721c884a23caf36fa?timestamp=2025-05-17T21%3A17%3A17.318Z&display=detailsType of Change
Testing
hatch fmt --linter
hatch fmt --formatter
hatch test --all
hatch run test-integ
Checklist
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.