Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OPIK-400] improve image truncation #595

Merged
merged 22 commits into from
Nov 11, 2024
Merged

Conversation

idoberko2
Copy link
Contributor

Details

Image truncation was introduced in #578. It had several issues:

  1. Wasn't working well for payloads with more than one image message
  2. Wasn't working at all for images lacking the data:image/... prefix
  3. Had a couple of tests that needed to be improved, see [OPIK-370] truncate images #578 (comment)

Issues

  • OPIK-400
  • OPIK-393

Testing

Improved E2E tests to cover all cases

@idoberko2 idoberko2 self-assigned this Nov 10, 2024
@idoberko2 idoberko2 requested a review from a team as a code owner November 10, 2024 12:50
@idoberko2 idoberko2 merged commit 89cae22 into main Nov 11, 2024
7 checks passed
@idoberko2 idoberko2 deleted the ido/opik-400-improve-truncation branch November 11, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants