-
Notifications
You must be signed in to change notification settings - Fork 66
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
feat: adopt hubble sdk on notebook login #576
Conversation
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
Co-authored-by: Alex Cureton-Griffiths <alexcg1@users.noreply.github.com>
…tuner into feat-notebook-login
@alexcg1 if you find this looks good please review again, otherwise will block us from merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
test failed blocked by : jina-ai/jina-hubble-sdk#83 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, only added one comment to be fixed.
"""Log in to Hubble account, initialize a client object | ||
and create a default experiment. | ||
|
||
:param force: If set to true, overwrite token and re-login. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the force parameter also to the login function. It also exists there.
docs/tasks/text-to-image.md
Outdated
@@ -11,7 +11,7 @@ For each product the dataset contains a title and images of multiple variants of | |||
|
|||
## Data | |||
Our journey starts locally. We have to {ref}`prepare the data and push it to the cloud <create-training-data>` and Finetuner will be able to get the dataset by its name. For this example, | |||
we already prepared the data, and we'll provide the names of training and evaluation data (`clip-fashion-train-data` and `clip-fashion-eval-data`) directly to Finetuner. | |||
we already prepared the data, and we'll provide the names of training and evaluation data (`fashion-eval-train-clip` and `fashion-eval-data-clip`) directly to Finetuner. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the train data are fashion-train-data-clip
. Update the snippet as well
📝 Docs are deployed on https://ft-feat-notebook-login--jina-docs.netlify.app 🎉 |
📝 Docs are deployed on https://ft-feat-notebook-login--jina-docs.netlify.app 🎉 |
The latest hubble sdk release brings finetuner sdk user experience improvement, see screenshot as follows:
After success: