-
Notifications
You must be signed in to change notification settings - Fork 988
Update Colab Base image to colab_20250219-060225_RC01 #1475
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
Changes from all commits
ba64a26
1448caa
435c29e
0fa466a
787f6e8
f955386
5753b3c
7705703
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
BASE_IMAGE=us-docker.pkg.dev/colab-images/public/runtime | ||
BASE_IMAGE_TAG=release-colab_20241217-060132_RC00 | ||
LIGHTGBM_VERSION=4.5.0 | ||
BASE_IMAGE_TAG=release-colab_20250219-060225_RC01 | ||
LIGHTGBM_VERSION=4.6.0 | ||
CUDA_MAJOR_VERSION=12 | ||
CUDA_MINOR_VERSION=2 | ||
CUDA_MINOR_VERSION=5 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,10 @@ | ||
import unittest | ||
|
||
import subprocess | ||
|
||
class TestTorchtune(unittest.TestCase): | ||
def test_help(self): | ||
ret_code = subprocess.run(["tune", "--help"]) | ||
self.assertEqual(0, ret_code.returncode) | ||
self.assertIsNone(ret_code.stderr) | ||
result = subprocess.run(["tune", "--help"], stdout=subprocess.PIPE) | ||
|
||
self.assertEqual(0, result.returncode) | ||
self.assertIsNone(result.stderr) | ||
self.assertIn("Download a model from the Hugging Face Hub or Kaggle Model Hub.", result.stdout.decode("utf-8")) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks! @neshdev Does this look sufficient to protect against loading the wrong tune lib in the future? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For context: This test was passing due to that fact we only check that the tune --help command went to completion with no specifics. Since ray and torch both have --help flags :lol-sob: , it passed. hence the added output check. |
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.
@rosbo Any concerns here, do we need a newer Keras?