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

Make soft over current timeout configurable #864

Conversation

wku12
Copy link
Contributor

@wku12 wku12 commented Sep 11, 2024

Describe your changes

Some meters are not able to achieve 7s refresh rate. Therefore it makes sense to make this parameter configurable

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@wku12 wku12 force-pushed the feat/make_soft_over_current_timeout_configurable branch 2 times, most recently from f58c22c to c05856e Compare September 11, 2024 13:32
@wku12
Copy link
Contributor Author

wku12 commented Sep 12, 2024

How can I fix the failing build?
Error: Input required and not supplied: token
Which token the CI needs?

Copy link
Contributor

@SebaLukas SebaLukas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@SebaLukas
Copy link
Contributor

Please update your branch with the latest main commit. Maybe the CI problem is fixed with this commit

@wku12 wku12 force-pushed the feat/make_soft_over_current_timeout_configurable branch from 8fd75a8 to 9bea3c0 Compare September 16, 2024 07:06
…thub.com:wku12/everest-core into feat/make_soft_over_current_timeout_configurable
@wku12 wku12 force-pushed the feat/make_soft_over_current_timeout_configurable branch from e517849 to 3c0d02e Compare September 16, 2024 09:51
@corneliusclaussen corneliusclaussen merged commit fd3bf0f into EVerest:main Sep 16, 2024
10 checks passed
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.

3 participants