Skip to content

[EAGLE-6653]: Fix TypeError for checkpoint size from environment variable#775

Merged
patricklundquist merged 1 commit intomasterfrom
fix-checkpoint-size-type-error
Sep 8, 2025
Merged

[EAGLE-6653]: Fix TypeError for checkpoint size from environment variable#775
patricklundquist merged 1 commit intomasterfrom
fix-checkpoint-size-type-error

Conversation

@patricklundquist
Copy link
Copy Markdown
Contributor

The CHECKPOINT_SIZE_BYTES environment variable is read as a string, which causes a TypeError when added to an integer. This change casts the value to an integer to prevent this error.

The `CHECKPOINT_SIZE_BYTES` environment variable is read as a string,
which causes a `TypeError` when added to an integer. This change
casts the value to an integer to prevent this error.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 5, 2025

Code Coverage

Package Line Rate Health
clarifai 43%
clarifai.cli 40%
clarifai.cli.templates 28%
clarifai.client 68%
clarifai.client.auth 67%
clarifai.constants 100%
clarifai.datasets 100%
clarifai.datasets.export 80%
clarifai.datasets.upload 75%
clarifai.datasets.upload.loaders 37%
clarifai.models 100%
clarifai.modules 0%
clarifai.rag 72%
clarifai.runners 53%
clarifai.runners.models 60%
clarifai.runners.pipeline_steps 44%
clarifai.runners.pipelines 80%
clarifai.runners.utils 63%
clarifai.runners.utils.data_types 72%
clarifai.schema 100%
clarifai.urls 60%
clarifai.utils 56%
clarifai.utils.evaluation 67%
clarifai.workflows 95%
Summary 62% (7810 / 12654)

Minimum allowed line rate is 50%

Copy link
Copy Markdown
Contributor

@wemoveon2 wemoveon2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alternatively, handle potential ValueError

s = os.environ.get('CHECKPOINT_SIZE_BYTES', '0')
if s.isdigit():
 checkpoint_size = int(s)
else:
 ...

@patricklundquist patricklundquist merged commit d1e9eb4 into master Sep 8, 2025
11 checks passed
@patricklundquist patricklundquist deleted the fix-checkpoint-size-type-error branch September 8, 2025 19:44
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