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

docs: update artifacts, lr, and results for tll experiment #599

Merged
merged 3 commits into from
Nov 8, 2022

Conversation

guenthermi
Copy link
Member

@guenthermi guenthermi commented Nov 4, 2022

  • update the artifacts in the tll example to store the images in blobs instead of tensors -> reduce the size of the training data from 1.5GB to ~150MB
  • change the learning rate of the TLL example to 1e-4
  • update the metrics in the TLL example (Colab and the README)

  • This PR references an open issue
  • I have added a line about this change to CHANGELOG

@guenthermi guenthermi marked this pull request as ready for review November 4, 2022 15:29
Copy link
Member

@bwanglzu bwanglzu left a comment

Choose a reason for hiding this comment

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

great job!

"* Additionally, we use {class}`~finetuner.callback.EvaluationCallback` for evaluation.\n",
"* Additionally, we use `finetuner.callback.EvaluationCallback` for evaluation.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this being changed?

Copy link
Member Author

Choose a reason for hiding this comment

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

It can not be displayed in the notebooks

Copy link
Contributor

@LMMilliken LMMilliken left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added size/s and removed size/m labels Nov 8, 2022
@github-actions
Copy link

github-actions bot commented Nov 8, 2022

📝 Docs are deployed on https://ft-docs-update-tll--jina-docs.netlify.app 🎉

@guenthermi guenthermi merged commit 5f665dd into main Nov 8, 2022
@guenthermi guenthermi deleted the docs-update-tll branch November 8, 2022 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants