-
Notifications
You must be signed in to change notification settings - Fork 406
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
feat: Improve the FeedbackDataset.__repr__ method #4103
feat: Improve the FeedbackDataset.__repr__ method #4103
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
📢 Thoughts on this report? Let us know! |
The URL of the deployed environment for this PR is https://argilla-quickstart-pr-4103-ki24f765kq-no.a.run.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kursathalat, I am not sure if the current formatting makes it more readable for me, could you try to play around with this a bit more and potentially align with the local FeedbackDataset
and include the metadata-properties
. Also, you left a unused import.
The problem with the current case is that when you are in a smaller screen, longer lines are wrapped improperly which looks messy. Here is the old version and new version (both looks proper on a bigger screen). If tabulate or any other library offers proper wrapping, we can iterate over the new one. |
closing this in favour of #4192 |
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Closes #3935
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)
How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference
tests
)Checklist
CHANGELOG.md
file (See https://keepachangelog.com/)