Roborock: various improvements to exposed sensors

[SLaks]

• Add clean_percent
• Expose options, device_class, suggested_display_precision on specific sensors for better homeassistant integration
• Also, add q revo to the list of devices that have a mop

[codecov]

Codecov Report

Attention: Patch coverage is 87.50000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.41%. Comparing base (8230bd4) to head (c03410a).

Files	Patch %	Lines
...o/integrations/roborock/vacuum/vacuumcontainers.py	84.61%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1914      +/-   ##
==========================================
- Coverage   81.42%   81.41%   -0.01%     
==========================================
  Files         193      193              
  Lines       18628    18636       +8     
  Branches     4041     4045       +4     
==========================================
+ Hits        15167    15173       +6     
- Misses       3179     3180       +1     
- Partials      282      283       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rytilahti]

LGTM, thanks for the PR, @SLaks! Did you mark this as a draft on purpose, or should this be merged right away?

[SLaks]

I marked this as draft because I couldn't test it due to rytilahti/homeassistant-xiaomi-ng#1.

Feel free to merge it; I may send followup PRs after #1916 merges to fix other issues or add other features (eg, I plan to add a sensor for WiFi signal strength).

[SLaks]

Update: I verified that the map dropdown works correctly. However, it resets to the first map in the list (which, incidentally, is the map the robot is physically in) after 10 seconds or so.

[rytilahti]

Looks good to me, tested on gen1 vac. We can either merge this and continue on subsequent PRs, or leave this open for the time being, your call.

[SLaks]

I pushed a WiFi strength sensor.

This is everything I wanted to add at the moment; I may send followups if I discover more issues

[rytilahti]

Thanks! I updated the title & description to be a bit more descriptive for the git logs, I hope that's fine with you.

[SLaks]

Of course Thanks!

…

On Tue, Mar 19, 2024, 9:16 PM Teemu R. ***@***.***> wrote: Merged #1914 <#1914> into master. — Reply to this email directly, view it on GitHub <#1914 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACMUJSOKJF3H5P6G6ILQH3YZDPOPAVCNFSM6AAAAABE2QB65OVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGE3TQMRYGMZTSMA> . You are receiving this because you were mentioned.Message ID: ***@***.***>