Skip to content

Conversation

daanelson
Copy link
Contributor

@daanelson daanelson commented Jun 10, 2025

Looks like there are two places where predictions can time out in safe push, one of which had no error text. Adding some so that it's easier for for folks to piece out why their safe push fails in the future.


Important

Add descriptive text to PredictionTimeoutError in predict() to include timeout duration and prediction index.

  • Error Handling:
    • In predict() function in predict.py, added descriptive text to PredictionTimeoutError to include the timeout duration and prediction index (if available).

This description was created by Ellipsis for cbc9ced. You can customize this summary. It will automatically update as commits are pushed.

@daanelson daanelson requested a review from andreasjansson June 10, 2025 04:18
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
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.

1 participant