Skip to content

Check if final_location is in /tmp in Kaggle environment#1201

Closed
dendarrion wants to merge 2 commits intounslothai:nightlyfrom
dendarrion:main
Closed

Check if final_location is in /tmp in Kaggle environment#1201
dendarrion wants to merge 2 commits intounslothai:nightlyfrom
dendarrion:main

Conversation

@dendarrion
Copy link
Copy Markdown
Contributor

No description provided.

Check whether path is in /tmp dir for Kaggle environment
@danielhanchen
Copy link
Copy Markdown
Contributor

@dendarrion Nice PR - actually if you want - it'll be better to directly override all Kaggle saving to /tmp - there is a check in save.py for IS_KAGGLE_ENVIRONMENT and if the folder /tmp exists, we should default to that especially for push_to_hub functions

Move temporary_location to /tmp in Kaggle
@danielhanchen danielhanchen changed the base branch from main to nightly October 30, 2024 07:39
@danielhanchen
Copy link
Copy Markdown
Contributor

Superseded by #1219

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