Skip to content

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

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

Documentation #780

wants to merge 6 commits into from

Conversation

stevemessick
Copy link
Contributor

@stevemessick stevemessick commented May 28, 2025

@stevemessick stevemessick marked this pull request as draft May 28, 2025 22:15
@stevemessick
Copy link
Contributor Author

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 GEMINI.md. I was surprised that worked. I still need to check it's work.

I'm considering adding .gemini and GEMINI.md to .gitignore.

@stevemessick stevemessick marked this pull request as ready for review June 3, 2025 17:41
@stevemessick stevemessick requested a review from jplotts June 3, 2025 17:42
@stevemessick
Copy link
Contributor Author

Removed .gemini and GEMINI.md from this PR. Added both to .gitignore.

@jplotts Ready for review now.

Copy link
Contributor

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`).
Copy link
Contributor

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.

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 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.
Copy link
Contributor

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).
Copy link
Contributor

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.

Comment on lines +40 to +41
* 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"`.
Copy link
Contributor

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.

Comment on lines +63 to +67
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`.
Copy link
Contributor

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.

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