Skip to content
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.

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

建议修改户号获取失败失败后的逻辑 #62

Closed
Ungs22 opened this issue Dec 9, 2024 · 7 comments
Closed

建议修改户号获取失败失败后的逻辑 #62

Ungs22 opened this issue Dec 9, 2024 · 7 comments

Comments

@Ungs22
Copy link

Ungs22 commented Dec 9, 2024

建议修改户号获取失败失败后的逻辑,当用户有多个户号时,某个户号获取失败时(目前看企事业户号一直提示获取错误),跳过该户号,继续执行下一个户号的数据获取,避免因一个户号始终获取不到导致后面的所有数据都一直更新不了。目前的逻辑是所有数据都获取成功后,才会执行后面的提交数据的步骤

@ARC-MX
Copy link
Owner

ARC-MX commented Dec 10, 2024

修复了一下,可以添加到环境变量中可忽略的ID中即可

@Ungs22
Copy link
Author

Ungs22 commented Dec 10, 2024

非常感谢大佬的修复!就是有个小建议,因为因为一些原因(主要是我也看不出啥问题),有的户号有时候能获取到数据,有时候获取不到。在原始的 _fetch 方法中,所有操作都在一个大的 try-catch 块中,这样一个户号获取失败,就导致了之前获取到的数据都被丢弃,也不会向Home Assistant提交数据。能不能细化下操作,一个户号重试三次获取失败后,就跳过该户号,获取下一个户号的信息。同时每个户号数据获取成功后,就向HomeAssistant提交一次数据。避免了因一个户号获取失败,导致所有数据都更新不了。非常感谢大佬!

@ARC-MX
Copy link
Owner

ARC-MX commented Dec 10, 2024

非常感谢大佬的修复!就是有个小建议,因为因为一些原因(主要是我也看不出啥问题),有的户号有时候能获取到数据,有时候获取不到。在原始的 _fetch 方法中,所有操作都在一个大的 try-catch 块中,这样一个户号获取失败,就导致了之前获取到的数据都被丢弃,也不会向Home Assistant提交数据。能不能细化下操作,一个户号重试三次获取失败后,就跳过该户号,获取下一个户号的信息。同时每个户号数据获取成功后,就向HomeAssistant提交一次数据。避免了因一个户号获取失败,导致所有数据都更新不了。非常感谢大佬!

这种情况,你要看看log里面报错在哪里,看看是啥数据读不出来,手动到官网看看有没有那个数据,如果官网有的话多半是你的操作延迟给的不够。如果官网没有这个数据的话那就是没办法。

@ARC-MX
Copy link
Owner

ARC-MX commented Dec 10, 2024

我有一个硬件是arm的小盒子,浏览器获取数据每个步骤都要延迟12s以上,不然获取不到数据,这是硬件能力不做没办法,解决办法只能是加大延迟RETRY_WAIT_TIME_OFFSET_UNIT

@Ungs22
Copy link
Author

Ungs22 commented Dec 10, 2024

我的延迟是默认的60s,我有5个户号,家庭、充电桩、企事业都有。我看日志是有个企事业的帐号一直提示失败,前两天试的时候就这个户号数据一直读不出来,但是今天开始又可以读出来了,充电桩的也没问题,所以感觉奇怪。我就自己拿cursor改了下代码逻辑,实在获取不到就直接跳过这个户号了,然后获取到一个户号就提交一次数据,目前可以正常运行

@ARC-MX
Copy link
Owner

ARC-MX commented Dec 10, 2024

嗯,确实是
你可以把你的代码合并进来

@Ungs22
Copy link
Author

Ungs22 commented Dec 10, 2024

已经提交合并了,但Cursor似乎把一些不相关的代码也改了,仅供大佬参考。经过实测在我本地运行是没问题的,但可能还有些Bug我没发现。再次感谢大佬的贡献

@ARC-MX ARC-MX closed this as completed Jan 5, 2025
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

No branches or pull requests

2 participants