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

Documentation updates #82

Merged
merged 4 commits into from
Oct 10, 2024
Merged

Documentation updates #82

merged 4 commits into from
Oct 10, 2024

Conversation

GeorgiosSmyrnis
Copy link
Contributor

@GeorgiosSmyrnis GeorgiosSmyrnis commented Oct 1, 2024

Currently included in this PR:

Copy link
Contributor

@afang-story afang-story left a comment

Choose a reason for hiding this comment

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

The documentation stuff looks good.

Is the fine-tuning stuff tested? Do we want to move it to a separate request?

@GeorgiosSmyrnis
Copy link
Contributor Author

The finetuning went through some basic testing, but agree that more could be done. I'll remove it from this PR.

@afang-story afang-story merged commit ac0c959 into main Oct 10, 2024
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