Skip to content

Improving silent errors on checking and submission#8

Open
ermolushka wants to merge 1 commit into
tensara:mainfrom
ermolushka:feat/submit-error-status-display
Open

Improving silent errors on checking and submission#8
ermolushka wants to merge 1 commit into
tensara:mainfrom
ermolushka:feat/submit-error-status-display

Conversation

@ermolushka
Copy link
Copy Markdown

pretty_print_submit_response was silently dropping

RUNTIME_ERROR, RATE_LIMIT_EXCEEDED, TOO_MANY_REQUESTS, TIME_LIMIT_EXCEEDED, MEMORY_LIMIT_EXCEEDED, SANDBOX_TIMEOUT, SANDBOX_OUTPUT_LIMIT, OUTPUT_LIMIT_EXCEEDED

responses. The spinner would just hang or show no output.

  • Added an explicit handler that surfaces the event type and error message.
  • Added a finished flag so that if the stream ends without hitting any terminal event the CLI prints "No response received from server." instead of exiting silently.
  • Added TOO_MANY_REQUESTS to the terminal-event matchers in pretty_print_checker_response and pretty_print_benchmark_response_v2, which were already handling the rest of the error family but missing this one.

Mentioned here as well

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