Skip to content

Add powerhub charging binary sensor and optional update intervals#77

Open
ItsRebaseTime wants to merge 1 commit into
m5stack:mainfrom
ItsRebaseTime:powerhub_charging_binary_sensor_and_update_intervals
Open

Add powerhub charging binary sensor and optional update intervals#77
ItsRebaseTime wants to merge 1 commit into
m5stack:mainfrom
ItsRebaseTime:powerhub_charging_binary_sensor_and_update_intervals

Conversation

@ItsRebaseTime
Copy link
Copy Markdown
Contributor

Added PowerHub binary sensor just to make a bit cleaner way to get the charging status without parsing the text sensor in lambdas.

Also noticed that there are no update interval configuration options for the sensors, so added those too.

@ItsRebaseTime ItsRebaseTime force-pushed the powerhub_charging_binary_sensor_and_update_intervals branch 2 times, most recently from 878f275 to f562474 Compare May 3, 2026 19:59
@ItsRebaseTime ItsRebaseTime force-pushed the powerhub_charging_binary_sensor_and_update_intervals branch from f562474 to 66f3a60 Compare May 3, 2026 20:05
@parker-int64
Copy link
Copy Markdown
Contributor

Thanks for the suggestion.

Refactoring the charge/vin status into a dedicated component was the right move. The original implementation focused on exposing raw register values, which—while technically accurate—is less relevant to the user than simply knowing the current state.

Although update_interval exist in pollingcomponent, somehow we are still using set_interval for each different sensors. (Different sensors may need different polling interval)

I'm thinking changing the structure to individual sub component.

@ItsRebaseTime
Copy link
Copy Markdown
Contributor Author

ItsRebaseTime commented May 16, 2026

@parker-int64 Could you give me a config yaml example of what you have in mind? I would be happy to do it

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