-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Documentation #780
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
base: main
Are you sure you want to change the base?
Documentation #780
Conversation
cc @jplotts I don't think this is finished. I'll delete the prompt file when it is. Notice how I handled the tutorial for creating model instances and model instance versions in I'm considering adding |
90c52a8
to
cbe5cb6
Compare
Removed @jplotts Ready for review now. |
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.
j/w - do these commands work? Last time I tried, they didn't seem to but maybe I did something wrong.
|
||
**Arguments:** | ||
|
||
* `<KERNEL>`: Kernel URL suffix (e.g., `kerneler/sqlite-global-default`). |
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.
nit - using a 9 year old kernel in a deprecated language doesn't feel like the best example. We should have some better examples, even if from the kerneler.
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.
Why is the file called model_instance_variations
if the commands are about model instance versions? I think the filename is wrong.
@@ -0,0 +1,182 @@ | |||
# Model Instances Commands | |||
|
|||
Commands for interacting with instances of Kaggle Models. A model instance typically represents a specific framework or variation of a parent model. |
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.
j/w - are framework and variation synonyms? If so, maybe worth clarifying that (or just picking one and sticking with it).
Before starting these tutorials, please make sure you have: | ||
|
||
1. Installed the Kaggle CLI. | ||
2. Downloaded your `kaggle.json` API token from your Kaggle account page (e.g., `https://www.kaggle.com/YOUR_USERNAME/account`) and placed it in the `~/.kaggle/` directory (or `C:\Users\<Windows-username>\.kaggle\` on Windows). |
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.
https://www.kaggle.com/YOUR_USERNAME/account
... the username
parameter is redirected out anyway. Probably best to simplify this to https://www.kaggle.com/settings
so no one is confused.
* Replace `"INSERT_TITLE_HERE"` with your desired dataset title, e.g., `"My Sample Dataset"`. | ||
* Replace `"INSERT_SLUG_HERE"` with a URL-friendly version of your title, e.g., `"my-sample-dataset"`. |
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.
Might be worth mentioning that these are related.
1. **Create or identify a kernel on Kaggle.com.** | ||
* Log in to kaggle.com. | ||
* Create a new notebook (or use an existing one). For this tutorial, let's assume its title is "My CLI Test Kernel". | ||
* Make a small change and **save a version** of the notebook on Kaggle.com (e.g., click "Save Version" and choose "Save & Run All (Commit)"). You cannot pull or push a kernel that is only in draft form. | ||
* Note the kernel slug from the browser's address bar. It will be something like `YOUR_USERNAME/my-cli-test-kernel`. |
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.
I feel like this implies that the kernel can't be created from the CLI. I think this should be changed to init
+ push
then pull
, or we'll get comments asking about how to create from the command line.
Some of GEMINI.md was adapted from https://github.com/harperreed/dotfiles/blob/master/.claude/CLAUDE.md.