Skip to content

DEEBOT T90 PRO functions. Only map has bugs like rooms. - #1701

Open
zhyf0610 wants to merge 14 commits into
DeebotUniverse:devfrom
zhyf0610:dev
Open

zhyf0610 wants to merge 14 commits into
DeebotUniverse:devfrom
zhyf0610:dev

Conversation

@zhyf0610

Copy link
Copy Markdown
Contributor

Almost Works.
Only Map functions like Rooms doesn't work.
Logs as follows:

Could not parse getMapSubSet for cr8hry: {'ret': 'ok', 'resp': {'body': {'code': 20003, 'msg': 'rcp not support'}, 'header': {'fwVer': '1.97.0', 'hwVer': '0.1.1', 'pri': 1, 'ts': '1783942345137', 'tzm': 480, 'ver': '0.0.1', 'wkVer': '0.1.54'}}, 'id': 'Fg4r', 'payloadType': 'j'}

Could not parse getMajorMap: {'body': {'code': 20003, 'msg': 'rcp not support'}, 'header': {'fwVer': '1.97.0', 'hwVer': '0.1.1', 'pri': 1, 'ts': '1783942338422', 'tzm': 480, 'ver': '0.0.1', 'wkVer': '0.1.54'}}

I suggest to merge after solving this problem.

zhyf0610 and others added 10 commits May 19, 2026 07:02
Added T30 Pro Plus device.
All buttons and switches tested.
Mainly edited from N20(viq3mw) and t30 pro omni(xco2fc) .
Almost Works.
Only Map functions like Rooms doesn't work.
Logs as follows:

Could not parse getMapSubSet for cr8hry: {'ret': 'ok', 'resp': {'body': {'code': 20003, 'msg': 'rcp not support'}, 'header': {'fwVer': '1.97.0', 'hwVer': '0.1.1', 'pri': 1, 'ts': '1783942345137', 'tzm': 480, 'ver': '0.0.1', 'wkVer': '0.1.54'}}, 'id': 'Fg4r', 'payloadType': 'j'}

Could not parse getMajorMap: {'body': {'code': 20003, 'msg': 'rcp not support'}, 'header': {'fwVer': '1.97.0', 'hwVer': '0.1.1', 'pri': 1, 'ts': '1783942338422', 'tzm': 480, 'ver': '0.0.1', 'wkVer': '0.1.54'}}
@zhyf0610

Copy link
Copy Markdown
Contributor Author

日志记录器: deebot_client.message
来源: runner.py:289
首次出现: 2026年7月18日 12:22:04 (84 次出现)
上次记录: 13:50:42

Handler for message getMapSubSet: {'code': 20003, 'msg': 'rcp not support'} returned no response. This is a bug should not happen. Please report it.
Handler for message getCleanInfo: {'code': 20003, 'msg': 'rcp not support'} returned no response. This is a bug should not happen. Please report it.
Could not parse getCleanInfo: {'body': {'code': 20003, 'msg': 'rcp not support'}, 'header': {'fwVer': '1.97.0', 'hwVer': '0.1.1', 'pri': 1, 'ts': '1784398768842', 'tzm': 480, 'ver': '0.0.1', 'wkVer': '0.1.54'}}
Further 'Could not parse getCleanInfo' entries will be logged at DEBUG level
Handler for message getMajorMap: {'code': 20003, 'msg': 'rcp not support'} returned no response. This is a bug should not happen. Please report it.

@zhyf0610

Copy link
Copy Markdown
Contributor Author

日志记录器: homeassistant
来源: /usr/src/homeassistant/homeassistant/runner.py:238
首次出现: 12:52:35 (1 次出现)
上次记录: 12:52:35

Error doing job: ClientConnectionResetError exception in shielded future (task: None)
Traceback (most recent call last):
File "/usr/local/lib/python3.14/site-packages/aiohttp/_websocket/writer.py", line 239, in _send_compressed_frame_async_locked
self._write_websocket_frame(
~~~~~~~~~~~~~~~~~~~~~~~~~~~^
(
^
...<8 lines>...
0x40,
^^^^^
)
^
File "/usr/local/lib/python3.14/site-packages/aiohttp/_websocket/writer.py", line 150, in _write_websocket_frame
raise ClientConnectionResetError("Cannot write to closing transport")
aiohttp.client_exceptions.ClientConnectionResetError: Cannot write to closing transport

@edenhaus edenhaus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please also fix the CI

Comment on lines +201 to +209
map=CapabilityMap(
cached_info=CapabilityEvent(CachedMapInfoEvent, [GetCachedMapInfo()]),
changed=CapabilityEvent(MapChangedEvent, []),
info=CapabilityExecute(GetMapInfoV2),
major=CapabilitySet(MajorMapEvent, [GetMajorMap()], SetMajorMap),
minor=CapabilityExecute(GetMinorMap),
multi_state=CapabilitySetEnable(
MultimapStateEvent, [GetMultimapState()], SetMultimapState
),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
map=CapabilityMap(
cached_info=CapabilityEvent(CachedMapInfoEvent, [GetCachedMapInfo()]),
changed=CapabilityEvent(MapChangedEvent, []),
info=CapabilityExecute(GetMapInfoV2),
major=CapabilitySet(MajorMapEvent, [GetMajorMap()], SetMajorMap),
minor=CapabilityExecute(GetMinorMap),
multi_state=CapabilitySetEnable(
MultimapStateEvent, [GetMultimapState()], SetMultimapState
),

as the map is not working from your PR description we should not set it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Well , map works. But room function doesn't work.
图片
图片

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ah okay then pls just fix the CI :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

CI fixed.

@edenhaus
edenhaus marked this pull request as draft August 1, 2026 14:04
@codecov

codecov Bot commented Aug 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.28%. Comparing base (5453eba) to head (02cd277).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1701      +/-   ##
==========================================
+ Coverage   96.26%   96.28%   +0.02%     
==========================================
  Files         161      162       +1     
  Lines        6399     6438      +39     
  Branches      368      368              
==========================================
+ Hits         6160     6199      +39     
  Misses        172      172              
  Partials       67       67              

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed

codspeed Bot commented Aug 1, 2026 •

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 226 untouched benchmarks


Comparing zhyf0610:dev (02cd277) with dev (5453eba)

Open in CodSpeed

@zhyf0610
zhyf0610 requested a review from edenhaus August 1, 2026 14:33
@zhyf0610

zhyf0610 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor Author
图片

Room function fixed.
But I don't know how to fix rust problems.

@zhyf0610
zhyf0610 marked this pull request as ready for review October 8, 2026 10:09

This branch has not been deployed

No deployments
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