Skip to content

Improved error logging#473

Merged
mogith-pn merged 1 commit intomasterfrom
DEVX-870-debug
Jan 9, 2025
Merged

Improved error logging#473
mogith-pn merged 1 commit intomasterfrom
DEVX-870-debug

Conversation

@mogith-pn
Copy link
Copy Markdown
Contributor

Why

  • When called from the upload_dataset with dataloader consists of image bytes the error is not catching properly. So added a failsafe checks and improved error logging.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 8, 2025

Code Coverage

Package Line Rate Health
clarifai 43%
clarifai.cli 65%
clarifai.client 68%
clarifai.client.auth 69%
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 9%
clarifai.runners.models 16%
clarifai.runners.utils 57%
clarifai.schema 100%
clarifai.urls 80%
clarifai.utils 77%
clarifai.utils.evaluation 67%
clarifai.workflows 94%
Summary 60% (3845 / 6357)

Minimum allowed line rate is 50%

Copy link
Copy Markdown
Contributor

@srikanthbachala20 srikanthbachala20 left a comment

Choose a reason for hiding this comment

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

LGTM

@mogith-pn mogith-pn merged commit 2ab31a8 into master Jan 9, 2025
@mogith-pn mogith-pn deleted the DEVX-870-debug branch January 9, 2025 09:39
@luv-bansal luv-bansal mentioned this pull request Jan 13, 2025
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