Replies: 1 comment 1 reply
-
Hey @luisoala, the feature is now implemented! Closing this discussion :) |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
i.e. right here
croissant/python/mlcroissant/mlcroissant/_src/datasets.py
Line 48 in 77fd5a7
for user ergonomics, would it not make more sense to just pass url to the json (e.g. https://github.com/mlcommons/croissant/blob/c3168c5d2b0203b92c2419128176bfb11c163370/datasets/coco2014/metadata.json) rather than having user first download the json and the pass the local file path
Beta Was this translation helpful? Give feedback.
All reactions