Skip to content

支持新风#87

Open
misaka4e21 wants to merge 42 commits intomypal:testfrom
misaka4e21:smallVAM
Open

支持新风#87
misaka4e21 wants to merge 42 commits intomypal:testfrom
misaka4e21:smallVAM

Conversation

@misaka4e21
Copy link
Copy Markdown

基于 test 分支,Merge 了 @lightrabbit 的 fork 并修复了一些在 HA 2025.1 中变化的 API 调用

mypal and others added 30 commits June 28, 2019 19:37
兼容网关版本02.10.00
B611应该使用 `SYS_GET_ROOM_INFO`而不是 `SYS_GET_ROOM_INFO_V1` 来获取房间信息
fix: 修正标准VAM的风速挡位数
feat: 把新风相关的日志设置为warning级别,方便HA上调试
@misaka4e21 misaka4e21 marked this pull request as draft January 27, 2025 05:00
@misaka4e21 misaka4e21 marked this pull request as ready for review January 27, 2025 05:08
@lightrabbit
Copy link
Copy Markdown

我之前改+测试的时候,忽略了D611和C611/B611的区别。目前看下来需要更改一下配置的部分,增加一个配置成D611的选项。

@llcc01
Copy link
Copy Markdown

llcc01 commented Feb 8, 2025

注意一下这里发起了两次 VentilationCapabilityQuery 分别查询标准VAM和smallVAM

https://github.com/misaka4e21/ha-dsair/blob/2839f7fdcff1c78c3af4b19b1849cd3012b17194/custom_components/ds_air/ds_air_service/decoder.py#L560-L567

而后一次的VentilationCapabilityQueryResult do处理逻辑中会覆盖前一次结果

https://github.com/misaka4e21/ha-dsair/blob/2839f7fdcff1c78c3af4b19b1849cd3012b17194/custom_components/ds_air/ds_air_service/decoder.py#L953

导致无法识别标准VAM。

另外一点是标准VAM的风量控制稍微有点问题。

上述问题可以参考我的两次 commit 修复

llcc01@b44c5d8

llcc01@6b8954e

@misaka4e21
Copy link
Copy Markdown
Author

感谢 @lightrabbit@llcc01 的修改意见
目前增加了 D611 的配置项,并且合并了 @llcc01 针对标准新风的修复。

@kamingchan 欢迎测试

@kamingchan
Copy link
Copy Markdown

感谢 @lightrabbit@llcc01 的修改意见 目前增加了 D611 的配置项,并且合并了 @llcc01 针对标准新风的修复。

@kamingchan 欢迎测试

测试了下面的场景:

  • 新风识别正常,名字能正常获取
  • 开关控制正常
  • 风速控制正常
  • 修改模式正常
  • 在其他平台修改后HA同步变动

🎉 感谢PR~

Copy link
Copy Markdown
Contributor

@xrh0905 xrh0905 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 to me(逃

本人没有D611也没有新风,无法测试,代码看起来没有问题。
该PR看起来肯定会和 #82 冲突,看哪个先合并吧。
以及个人感觉每多一个网关多一个is_x611有点不太方便,最好直接抽象成一个型号字段?

@caibinqing
Copy link
Copy Markdown
Contributor

Looks good to me(逃

本人没有D611也没有新风,无法测试,代码看起来没有问题。 该PR看起来肯定会和 #82 冲突,看哪个先合并吧。 以及个人感觉每多一个网关多一个is_x611有点不太方便,最好直接抽象成一个型号字段?

先合这边呗,我会抽空解决冲突的。

@lightrabbit
Copy link
Copy Markdown

我这的配置如下:
网关: D611
新风: Small VAM

目前测试下来,网关配置选择D611后。
新风能正常显示,新风内传感器的信息也都能正常显示。
空调和外置传感器的数据也显示正常。

@misaka4e21 misaka4e21 changed the title [待测试] 支持新风 支持新风 Feb 16, 2025
@devkaiwang
Copy link
Copy Markdown

devkaiwang commented Nov 30, 2025

用这个PR分支,两台新风机无法识别。网关型号是DTA117D611,新风型号是VAMS250AV1。使用@llcc01 的分支,亦是如此。

@devkaiwang
Copy link
Copy Markdown

这个项目还有人在用/维护吗?

@lightrabbit
Copy link
Copy Markdown

这个项目还有人在用/维护吗?

这个分支我目前一直在用,我的网关型号和你的相同,新风型号是 VAMS150AV1 目前用下来都没什么问题。
你所说的无法识别是没找到新风,还是别的问题?

@devkaiwang
Copy link
Copy Markdown

这个项目还有人在用/维护吗?

这个分支我目前一直在用,我的网关型号和你的相同,新风型号是 VAMS150AV1 目前用下来都没什么问题。 你所说的无法识别是没找到新风,还是别的问题?

设备列表里有空调和空气传感器,但没有新风。

@devkaiwang
Copy link
Copy Markdown

devkaiwang commented Dec 22, 2025

我家有两个新风。
查了一下,发现VentilationCapabilityQueryResult.load_bytes中,两个房间之间有一个额外的字节,需要读取,否则新风解析失败。修改之后,两台新风正常控制。
PR: misaka4e21#1

@misaka4e21 @lightrabbit 麻烦review!谢谢!

另外,在墙上的控制面板和大金App中,新风的风量只有两档,但现在集成里有0%, 25%, 50%, 75%, 100%五档。这是个Bug,还是我家新风型号的关系?

@devkaiwang
Copy link
Copy Markdown

devkaiwang commented Jan 6, 2026

为了更多的人可以使用到这个插件,是不是应该更活跃地维护起来?我这边两台新风已经可用,虽然有我上一个评论里的问题。

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.

9 participants