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

fix: don't count time with any job popped as "no job" time; add way to suppress job time warnings #301

Merged

tazlin merged 5 commits into main from hotfix-no-job-time

Sep 29, 2024

Member

tazlin commented Sep 29, 2024

Sets default warning time for no jobs popped to 30 minutes combined.
Clarifies the meaning of the warning associated with this message. It now reads:

Your worker spent more than {minutes_allowed_without_jobs} minutes combined throughout this session ({cur_session_minutes:.2f} minutes) without jobs. This may be due to low demand. However, offering more models or increasing your max_power may help increase the number of jobs you receive and reduce downtime.

Setting suppress_speed_warnings: true in your config will suppress this message from appearing.

tazlin added 5 commits

September 29, 2024 16:30


          fix: don't count no-job-time if a job is already popped

109cd8d


          fix: add way to suppress no-job time messages

05db3af


          fix: missing type annotation for minutes_allowed_without_jobs

af24482


          fix: clarify the meaning of no-jobs-time message

17dc9d7


          chore: version bump

cc8645f

tazlin merged commit da894e8 into main

3 checks passed

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

Labels

None yet