Skip to content
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

[LLM] Add S3 details to readme #674

Merged
merged 2 commits into from
Jul 25, 2023

Conversation

mikolajblaz
Copy link
Contributor

No description provided.

@mikolajblaz mikolajblaz requested a review from a team as a code owner July 13, 2023 17:26
@github-actions
Copy link

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

@nv-rborkar
Copy link
Contributor

Thanks @mikolajblaz .

@sgpyc , @erichan1 Can you please review this urgent MR? This updates the dataset location to the MLC S3 bucket location. We have many requesters trying to access LLM dataset and we should enable them asap.

@nathanw-mlc
Copy link
Member

nathanw-mlc commented Jul 25, 2023

@nv-rborkar Are you good with me forcing this PR through to make clear where to get the datasets at the moment?

@TheKanter TheKanter self-assigned this Jul 25, 2023
@TheKanter TheKanter self-requested a review July 25, 2023 19:17
Copy link
Contributor

@TheKanter TheKanter left a comment

Choose a reason for hiding this comment

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

I hope this works, if not. Then please blame @nv-rborkar

@nv-rborkar nv-rborkar merged commit d0012fd into mlcommons:master Jul 25, 2023
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants