Skip to content

Conversation

@glesperance
Copy link
Contributor

Printing DSPy Examples containing images creates problems because they flood the console / notebook outputs with massive base64-encoded strings.

Base64-encoded images can be thousands of characters long, making it nearly impossible to read other important output or debug information.

@okhat
Copy link
Collaborator

okhat commented Nov 13, 2024

Awesome. Thanks a lot @glesperance !

@okhat
Copy link
Collaborator

okhat commented Nov 13, 2024

@glesperance one test failed, mind checking and adjusting the code or the test?

@glesperance
Copy link
Contributor Author

@okhat fixed the broken test & added an additional one for dspy.Image handling

@okhat
Copy link
Collaborator

okhat commented Nov 16, 2024

@isaacbmiller can we merge this now? or does it conflict with the PR on Images in types?

@isaacbmiller isaacbmiller merged commit 33cf45f into stanfordnlp:main Nov 16, 2024
4 checks passed
@glesperance
Copy link
Contributor Author

Just saw that we went to main. Thanks everyone!

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.

3 participants