Skip to content

Conversation

@pdurbin
Copy link
Member

@pdurbin pdurbin commented Jun 6, 2025

This is the same link shown at https://mlcommons.org/working-groups/data/croissant/

Perhaps the spec could also be linked in the first paragraph. 🤷 This is a a start, at least. I was showing a friend the README and he asked where he could find the spec. We went to https://mlcommons.org/croissant and scrolled down to find it.

I'm aware that "Specification" also appears just below under "GitHub Repo". Perhaps this could link to the specs in progress? 🤷

@pdurbin pdurbin requested a review from a team as a code owner June 6, 2025 15:21
@github-actions
Copy link

github-actions bot commented Jun 6, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅


## Resources

- [Specification](https://docs.mlcommons.org/croissant/docs/croissant-spec.html)
Copy link
Contributor

Choose a reason for hiding this comment

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

Format Specification?

Copy link
Member Author

Choose a reason for hiding this comment

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

Should I make this change? Please feel free to push to my branch!


- [Specification](https://docs.mlcommons.org/croissant/docs/croissant-spec.html)
- [Github Repo](https://github.com/mlcommons/croissant)
- Specification
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I think we could remove this line

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we could add here something like "[mlcroissant code](link to python/mlcroissant)" instead

Copy link
Member Author

Choose a reason for hiding this comment

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

As you wish. Please feel free to push a change.

Copy link
Contributor

@ccl-core ccl-core left a comment

Choose a reason for hiding this comment

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

Thanks!

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.

2 participants