Skip to content
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.

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

pbio/sys/hmi: Reduce time needed to power off hub. #250

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Conversation

laurensvalk
Copy link
Member

@laurensvalk laurensvalk commented Jun 7, 2024

Reduces the power off time from 3 to 2 seconds. This has been requested occasionally. I think this is OK.

If it makes certain use cases harder, like long press menus, we could now quite easily add a user setting to make this timeout configurable due to pybricks/support#1622 now being implemented.

@coveralls
Copy link

Coverage Status

coverage: 56.592%. remained the same
when pulling c13bcda on storage
into 83e372a on master.

@laurensvalk
Copy link
Member Author

This was requested among others by @biasedlogic.

@coveralls
Copy link

Coverage Status

coverage: 56.592%. remained the same
when pulling 42f10e1 on storage
into 5a24099 on master.

@laurensvalk laurensvalk merged commit 42f10e1 into master Jul 22, 2024
38 checks passed
@laurensvalk
Copy link
Member Author

We will merge this since there haven't been any objections for a month. If anyone needs a longer shutdown-time out, please feel free to open a new issue. Then we make make the duration a user-configurable setting. Perhaps as a second argument to the existing set_stop_button method.

@laurensvalk laurensvalk deleted the storage branch July 22, 2024 11:42
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