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.

Sign up for GitHub

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

Jump to bottom

Minor UX improvements #138

Merged

chriso merged 2 commits into main from minor-ux

Mar 26, 2024

Contributor

chriso commented Mar 26, 2024

Two improvements I noticed:

I moved the batch helper from dispatch.fastapi.Dispatch to the dispatch.function.Registry base class, so that it can be used when you construct a remote endpoint registry (Remote endpoints #134)
when quitting the python -m dispatch.test server with SIGINT / Ctrl+C, previously it would print a stack trace from where the KeyboardInterrupt was raised. Now it prints a nice single-line exit message

chriso added 2 commits

March 26, 2024 13:24


          Move batch helper to base class, so it can be used with remote endpoints

8cc9289


          Don't print stack trace on KeyboardInterrupt

d183f51

chriso requested review from pelletier, achille-roussel and Pryz

March 26, 2024 03:28

achille-roussel approved these changes

View reviewed changes

Contributor

achille-roussel left a comment

👏

chriso merged commit b3f8af3 into main

3 checks passed

chriso deleted the minor-ux branch

March 26, 2024 03:34

chriso self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet