MatterVN / ModbusTCP2MQTT

Sungrow & SMA Solar Inverter addon for Home Assistant using mobusTCP
MIT License
69 stars 52 forks source link

Modified abs reference of pycryptodomex==3.11.0 to nearest compatible. #73

Closed loucksg closed 8 months ago

loucksg commented 11 months ago

aiming to fix compatibility problem with the python crypto library install within the docker container.

kilador commented 10 months ago

I tried your solution: changing "==" by "~=" but still get the same error. Any other idea?

loucksg commented 10 months ago

Hi Kilador,Ok apparently the packages are no more and HA does not allow us to build. There are a couple streams of work on it, there is a sungather discord chat we’re working over there I’ll update this ticket with the results as well.GuyGuy's mobileOn 16 Jan 2024, at 12:03 pm, kilador @.***> wrote: I tried your solution: changing "==" by "~=" but still get the same error. Any other idea?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

oleg-d commented 9 months ago

Can this be merged and new release made with this fix? There's quite a few people waiting to try this integration out as the GoSungrow one has been broken.

roabyd commented 9 months ago

@oleg-d unfortunately this PR would not fix this issue (I cloned the repo and tried updating it myself). It looks like there is a bigger issue here as per @loucksg's comment. Here's hoping a fix can be found soon

roabyd commented 9 months ago

image