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

Updated hyperparams to Unsloth recommended settings #3

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

Northwind01
Copy link

@Northwind01 Northwind01 commented Oct 15, 2024

The original version of the notebook has very low performance on our test bot (1/14 passing tests). This PR updates the hyper-parameters of fine-tuning to the settings from the Unsloth recommendations in their tutorial. These hyper-parameters worked much better for the test bot (13/14 tests passing).

Additionally, we are adding an environment.yml file to ensure reproducibility of the conda environment we used to perform fine-tuning in the notebook.

@Northwind01 Northwind01 requested a review from a team as a code owner October 15, 2024 10:10
@Northwind01 Northwind01 requested review from rasa-jmac and dakshvar22 and removed request for rasa-jmac October 15, 2024 10:10
cmd_gen_finetuning.ipynb Outdated Show resolved Hide resolved
Copy link
Contributor

@rafaeljcdarce rafaeljcdarce left a comment

Choose a reason for hiding this comment

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

I have some questions

cmd_gen_finetuning.ipynb Outdated Show resolved Hide resolved
cmd_gen_finetuning.ipynb Outdated Show resolved Hide resolved
cmd_gen_finetuning.ipynb Show resolved Hide resolved
cmd_gen_finetuning.ipynb Outdated Show resolved Hide resolved
@dakshvar22 dakshvar22 removed their request for review October 17, 2024 13:31
@Northwind01 Northwind01 removed the request for review from djcowley October 18, 2024 10:53
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.

3 participants