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

Clean up train README #392

Merged
merged 32 commits into from
Jun 30, 2023
Merged

Clean up train README #392

merged 32 commits into from
Jun 30, 2023

Conversation

jacobfulano
Copy link
Contributor

No description provided.

@jacobfulano jacobfulano changed the title Add hyperlink Add hyperlink to README Jun 29, 2023
@jacobfulano jacobfulano changed the title Add hyperlink to README Clean up train README Jun 29, 2023
scripts/train/README.md Outdated Show resolved Hide resolved
@jacobfulano jacobfulano requested a review from vchiley June 29, 2023 21:14
@jacobfulano jacobfulano marked this pull request as draft June 29, 2023 21:14
scripts/train/README.md Outdated Show resolved Hide resolved
@vchiley vchiley self-requested a review June 30, 2023 00:23
@jacobfulano jacobfulano marked this pull request as ready for review June 30, 2023 01:40
@jacobfulano
Copy link
Contributor Author

I have tried to make the README for the train folder somewhat self explanatory and consistent. I honestly think there is too much detail in there that might make it confusing for users to easily consume. In the next release I think pretraining and finetuning instructions should be in separate READMEs

scripts/train/README.md Outdated Show resolved Hide resolved
scripts/train/README.md Outdated Show resolved Hide resolved
scripts/train/README.md Outdated Show resolved Hide resolved
@vchiley vchiley self-requested a review June 30, 2023 02:56
scripts/train/README.md Outdated Show resolved Hide resolved
scripts/train/README.md Outdated Show resolved Hide resolved
@vchiley vchiley self-requested a review June 30, 2023 03:15
@vchiley vchiley merged commit 33aeb79 into main Jun 30, 2023
9 checks passed
@dakinggg dakinggg deleted the jacobfulano-patch-1-1 branch October 11, 2023 21:32
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