Replies: 23 comments 1 reply
-
Thanks for sharing this. This will have implications for the driver as it is uses the Victron MQTT libraries. I will monitor Victron's project and plan accordingly. Watch this space. |
Beta Was this translation helpful? Give feedback.
-
Thanks, because the driver doesnt work with VenusOS 3.10 anymore. |
Beta Was this translation helpful? Give feedback.
-
Please provide a log file showing any errors. |
Beta Was this translation helpful? Give feedback.
-
Hi guys, the change from Mosquitto to FlashMQ includes |
Beta Was this translation helpful? Give feedback.
-
@mpvader, Thanks for confirmation but are you Deprecating or removing? Deprecating would mean it should continue to work for a while longer. |
Beta Was this translation helpful? Give feedback.
-
ah thanks. Removing. There only one service can listen to a port. So what we did per v3.10~12, which is change who listens to the local websockets port from mosquitto to flashmq, is what we'll be doing more coming time. |
Beta Was this translation helpful? Give feedback.
-
ps. note that our MQTT API doesn't change much. There are some very nice improvements, especially related to the keepalive and supressing a resend of all topics; and otherwise all stays the same. Its just faster, and causes less CPU load on the GX device. And is done via FlashMQ, which is developed by one of our own core team members in his spare time - which is maybe business wise not that important but personally I'm quite proud of that/him.! |
Beta Was this translation helpful? Give feedback.
-
Any new progess-info for an separate flashMQ-version? |
Beta Was this translation helpful? Give feedback.
-
It's unlikely I'll be able to make any significant updates for a few months. If you have already experienced an error please share log file, as requested previously. |
Beta Was this translation helpful? Give feedback.
-
v. 3.10 ~27
|
Beta Was this translation helpful? Give feedback.
-
That error appears to be the problem related to the missing YAML library and not specifically the new version of Venus OS. |
Beta Was this translation helpful? Give feedback.
-
with VenusOS v3.0 it works without any error. Edit: reinstalling dbus-mqtt-devices does the trick! 👍🏻 |
Beta Was this translation helpful? Give feedback.
-
That's as maybe but others are also reporting this issue with the missing PYYaml library on Venus 3.0. PYYaml is not included by Victron in any version. |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
Hi all, there are now people thinking that there are issues with running this code (dbus-mqtt-devices) in combination with FlashMQ. See above link by disaster123 for an example. Having read the thread above, this is all just a suspicion and vague. I don't see any problem related to FlashMQ. And, note that I also don't expect any problem. |
Beta Was this translation helpful? Give feedback.
-
@freakent , am I right in concluding that this repo is expected to keep working fine as long as our dbus-mqtt project is installed? What/where is the depedency of dbus-mqtt-devices on dbus-mqtt? |
Beta Was this translation helpful? Give feedback.
-
Hi @mpvader , thanks for keeping an eye on this project :-) I am not aware of any issue using this driver with FlashMQ, although I intend to do more testing. To be clear, from what I know of FlashMQ I do NOT currently see the need to create a seperate or new version of this driver as a result. This driver has had an issue recently with a PyYaml dependency that started roughly the same time as the Venus 3.10 Beta. Some users wrongly assumed the issue was caused by Venus 3.10, when in fact it was an outdated dependency on PyYaml. This issue has now been addressed in the latest relesse. In the longer term I am working on a new version that removes the PyYaml dependency completely. The driver does today have a minor dependency on dbus-mqtt. To simplify coding my MQTTDeviceManager class extends the MqttGObjectBridge class in the dbus-mqtt module. During installation, the setup script creates a soft link from /opt/victronenergy/dbus-mqtt to include this module in the python module search path for the driver. In the future I could either just include the source for this class directly in my own source tree or write my own version. Would you consider including the functionality of this driver in VenusOS directly? It would be more efficient to have a single driver for any external MQTT devices instead of one for each, when each driver more or less does the same thing? Thanks again. |
Beta Was this translation helpful? Give feedback.
-
@freakent , Curious if you will be testing against 3.20~17 now that it is released with the new html5 app and flashmq? I have it installed and tanks do not show up correctly in flashmq on port 1884 but they appear correctly in mosquito mqtt on port 1883. The tanks do not show up in the new html5 app. The new html5 |
Beta Was this translation helpful? Give feedback.
-
Thanks for the heads up. I'm juggling lots of projects at the moment, but I'll try to take a look next week. It's strange that you don't see it in the new HTML app. Does it appear in VRM? |
Beta Was this translation helpful? Give feedback.
-
Yes. Tanks and Temps show correctly in VRM Thanks!! |
Beta Was this translation helpful? Give feedback.
-
@freakent I believe I discovered the issue. In short, the device_service.py was setting the dbus path "/DeviceInstance as a string value. I edited your code to cast it as an integer -- dbus_service.add_path('/DeviceInstance', int(self.device_instance)) And while browsing flashmq on port 1884. dbus-flashmq was indexing the devices incorrectly as it would only publish one of the devices with an mqqt path of 0 when the DeviceInstance was 4 as seen in this screenshot. Hope this helps All the best |
Beta Was this translation helpful? Give feedback.
-
@gharshaw Gene, thanks for the info but what version of the code is this in? Please can you post the block of code you changed? If this against the Beta version please can you post feedback there? |
Beta Was this translation helpful? Give feedback.
-
@freakent I'm using your 0.6.2 driver on CCGX v3.20~17. Edited file device_service.py line 82 |
Beta Was this translation helpful? Give feedback.
-
#1098
Beta Was this translation helpful? Give feedback.
All reactions