Skip to content

feat: Q7 Get battery level #790

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

Merged
Lash-L merged 6 commits into from
Mar 19, 2026
Merged

feat: Q7 Get battery level #790

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d4805d5
feat: Q7 Get battery level
RaddedMC Mar 17, 2026
2f5cc5d
feat: Q7 Get battery level
RaddedMC Mar 17, 2026
480fb8b
Merge branch 'main' of https://github.com/RaddedMC/python-roborock
RaddedMC Mar 17, 2026
2d7cf36
Merge branch 'main' of https://github.com/RaddedMC/python-roborock
RaddedMC Mar 17, 2026
dec8cc4
Merge branch 'main' of https://github.com/RaddedMC/python-roborock
RaddedMC Mar 18, 2026
d07916a
Run lint
RaddedMC Mar 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions roborock/devices/traits/b01/q7/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,19 @@ async def set_clean_path_preference(self, preference: CleanPathPreferenceMapping
async def set_repeat_state(self, repeat: CleanRepeatMapping) -> None:
"""Set the cleaning repeat state (cycles)."""
await self.set_prop(RoborockB01Props.REPEAT_STATE, repeat.code)

async def get_battery_level(self) -> int | None:
Copy link
Copy Markdown
Collaborator

@Lash-L Lash-L Mar 17, 2026

Choose a reason for hiding this comment

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

Is this the same quantity we get back from the main get props?

Copy link
Copy Markdown
Contributor Author

@RaddedMC RaddedMC Mar 17, 2026

Choose a reason for hiding this comment

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

Yes it is. I don't quite understand why but this corresponds to the current battery level at least on my vacuum.

Copy link
Copy Markdown
Collaborator

@Lash-L Lash-L Mar 17, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Then we should actually be able to avoid adding this function at all and just grab the quantity property on the home assistant side I think? Does that seem to align with your understanding? Assuming you are adding this functionality for HA

"""Get the battery level (%)"""
level_query = await self.send(
command=RoborockB01Q7Methods.GET_PROP,
params={
"property": ["quantity"]
}
)
try:
return level_query['quantity']
Copy link
Copy Markdown
Collaborator

@Lash-L Lash-L Mar 17, 2026

Choose a reason for hiding this comment

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

what else is in the response dict?

except:
return None

async def start_clean(self) -> None:
"""Start cleaning."""
Expand Down
Loading