fix: push_to_huggingface parsing of UUIDs

[alvarobartt]

Description

Due to the recent constraint for the FeedbackRecord.user_ids to be UUIDs instead of strs, the conversion to dict() was failing, as the UUIDs are not JSON-serializable, which was leading to some issues when trying to add the example record to the DatasetCard via the dict() conversion.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

• Re-run unit tests to pass when generating the DatasetCard for FeedbackDatasets

Checklist

• I have merged the original branch into my forked branch
• I added relevant documentation
• follows the style guidelines of this project
• I did a self-review of my code
• I made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

[gabrielmbmb]

Hey Alvaro, LGTM for now :) maybe we could think to add a custom type like StrUUID https://docs.pydantic.dev/latest/usage/types/#classes-with-__get_validators__, so when calling .dict() we get an str instead of a UUID.

Using json.loads(model.json()) makes the trick, but it's a bit inefficient as we're serializing/deserializing the whole model.

[alvarobartt]

Hey Alvaro, LGTM for now :) maybe we could think to add a custom type like StrUUID https://docs.pydantic.dev/latest/usage/types/#classes-with-__get_validators__, so when calling .dict() we get an str instead of a UUID.

Using json.loads(model.json()) makes the trick, but it's a bit inefficient as we're serializing/deserializing the whole model.

Yes, I agree with this, I'll tackle next release, this was a patch proposed by @frascuchon but since it's just the UUIDs and the datetimes, and we're not pushing datetimes the solution above or a custom encoder would do the work! Thanks