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

Setting Kaggle Credentials #288

Closed
wants to merge 1 commit into from

Conversation

ayyucedemirbas
Copy link

Description

The existing code in the notebook for setting Kaggle credentials uses the Colab-specific userdata.get API, which may not work outside Colab. The added method works in different environments. It simply creates a .kaggle directory and writes the API token to a kaggle.json file.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

The added cell is used to set up the Kaggle credentials to download the Paligemma model from Kaggle on Google Colab. It also works on the local machine.

Any specific deployment considerations

N/A

Docs

  • Docs updated? What were the changes:

The existing code in the notebook for setting Kaggle credentials uses the Colab-specific userdata.get API, which may not work outside Colab. The added method works in different environments. It simply creates a .kaggle directory and writes the API token to a kaggle.json file.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@LinasKo
Copy link
Collaborator

LinasKo commented Nov 27, 2024

Hi @ayyucedemirbas,

Google Colab is our platform of choice, which most users are familiar with. You're right - the API here is specific to it; however, we strive to keep our notebooks as simple as possible, with only the code essential to illustrate the example.

I'll be closing this PR, but thank you for the contribution nonetheless!

@LinasKo LinasKo closed this Nov 27, 2024
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