Skip to content

Dataverse sample in croissant format #232

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Dataverse sample in croissant format #232

wants to merge 4 commits into from

Conversation

4tikhonov
Copy link

This example converted from OAI-ORE export of dataset from Dataverse data network https://dataverse.org

@4tikhonov 4tikhonov requested a review from a team as a code owner September 27, 2023 21:39
@github-actions
Copy link

MLCommons CLA bot:
Thank you very much for your submission, we really appreciate it. Before we can accept your contribution, we ask that you sign the MLCommons CLA (Apache 2). Please use this [Google form] (https://forms.gle/Ew1KkBVpyeJDuRw67) to initiate authorization. If you are from an MLCommons member organization, we will request that you be added to the CLA. If you are not from a member organization, we will email you a CLA to sign. For any questions, please contact [email protected].
0 out of 1 committers have signed the MLCommons CLA.
❌ @4tikhonov
You can retrigger this bot by commenting recheck in this Pull Request

marcenacp
marcenacp reviewed Oct 10, 2023
View reviewed changes
datasets/dataverse/metadata.json
{
"@type": "sc:FileObject",
"name": "metadata",
"contentUrl": "https://zenodo.org/record/6615455/files/pass_metadata.csv",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the PASS dataset. Should you adapt it to a dataset from https://dataverse.nl?

marcenacp
marcenacp requested changes Oct 10, 2023
View reviewed changes
Copy link
Contributor

@marcenacp marcenacp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@4tikhonov 4tikhonov requested a review from a team as a code owner April 28, 2025 18:22
benjelloun
benjelloun reviewed Apr 29, 2025
View reviewed changes
Copy link
Contributor

@benjelloun benjelloun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please put the dataset under the datasets/1.0 directory, and make sure the Croissant is valid according to the 1.0 spec by using the mlcroissant validator.

datasets/dataverse/metadata.json
"includes": "ml:includes",
"isEnumeration": "ml:isEnumeration",
"jsonPath": "ml:jsonPath",
"ml": "http://mlcommons.org/schema/",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We generally use "cr" as a prefix for Croissant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjelloun benjelloun benjelloun left review comments

@marcenacp marcenacp marcenacp requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@4tikhonov @benjelloun @marcenacp