-
-
Notifications
You must be signed in to change notification settings - Fork 113
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
Problem with system with more than 4 mixers. #294
Comments
wow, that's one hell of a system. Could you send over the devices you have? http://ems-esp.local/api/system |
I'm also thinking of extending the customization service we have in v3.4 so that each device can be renamed |
{ |
One thing is a simple naming issue, we have only names for h1-hc4, as the thermostats only have 4 heatingcircuits. (we need to insert TAG_HC5, -TAG_HC8) |
can you make a log from the telegrams (more than a minute). Can you tell me please, what should I do exactly, Iam first day user :))) |
2022-01-15 19:09:34.935 D 256: [telegram] Sending read Tx [#108], telegram: 8B 90 FF 00 20 01 9C |
Good, but use log level |
ok |
0x20-0x23 and 0x25 - hc |
EMS Devices screen: |
Ok, looks like the pool behave like a normal heating circuit and telegram 0x2DB is right:
Then we only need to add the names. The telegram contains no measured temperature, flow-setpoint is 70°C, @proddy should i make a pr to 3.4, or dev-branch? |
I have made a little video. First, the sensors receive data from 0x28, and then from 0x2 video_2022-01-15_19-43-00.mp44 |
I've added to proddys v3.4alpha, please check if this solves the issue. After uploading the bin you need to refresh the page in your browser. |
thanks, uploading |
Now I see all mixers in HA, thank you! This firmware is a very big progress in your work! |
thanks @MichaelDvP for the quick fix |
Next step is to add the heating circuits to the thermostat. I've found on NorbertS' list that this thermostat have 8 ciircuits in row. I'm curious if the buffers for web/mqtt are large enought for all these circuits. Please check dashboard for thermostat, mqtt and http://ems-esp.local/api/thermostat |
{ |
Looks good, only hc1 have to few values, wait a minute to allow emsesp to read all values. Does web-dashboard also show all values for all hc? Also check Settings -> Customization for the thermostat. |
Yes? dashboard shows all. Customization ok. |
After restart it always take some minutes to collect all data from bus. |
the API call to http://ems-esp.local/api/thermostat uses a large buffer (EMSESP_JSON_SIZE_XXLARGE_DYN in WebAPIService.cpp) so I would expect to see all the hc1 values in that call |
I have changed Enum-format from Index to Value and now I have: Boiler entities 43 (was 56), thermostat entities 133 (was 133). But http://ems-esp.local/api/thermostat now shows "null", customization is empty for all devices. I try to watch entities by tapping on device and the list of sensors do not appear. |
I do all these actions with a delay of more than 5 minutes |
Thanks, i see the problem.
|
This is the quick solution, i think now all values should be there. I've added the suggested warning for mqtt overflow. |
thanks @MichaelDvP for those fixes |
Good job, thank you! |
@Shmul1 Have you already checked the mqtt-output? |
Yes, of course, I sent the log back on Sunday. The firmware works well, but there is one more thing. In your firmware, warm water is prepared only through a circuit in the boiler, and in my case through a separate circuit "wwc1", so the sensors associated with warm water do not work. Is it possible to fix this? |
It need some work, but i think it's possible. The wwc has two telegrams: |
Ok, I will do it today |
I have changed different settings and made the lo |
Please check if this is enough |
I see changing values in the unknown telegram 0x313 on offsets 4, 5, 10 but as written before i need a reference what value has changed and to what value is has changed. First change was offset 10 to 0x41, maybe a temperature to 65°C, or a bitfield bits 0 and 7 set? |
Maybe it's better to make a video and make a log at the same time? |
output.compress-video-online.com.1.mp4 |
Ok, here are some more values. Some settings (like disinfect day, etc. are already in thermostat. Some values you have skipped, and values you have shown on video but not changed are not logged. But it's a good idea with the video. |
Good afternoon. Unfortunately, I had to leave on a business trip before Saturday and now I can only do everything remotely. I'm ready to make more videos on Saturday and Sunday. I installed the firmware, there is integration in HA, but I can test it in more detail only at night. Thank you for your work |
@MichaelDvP this was a bug. I was sorting the vector but always returning the last hc in the list. It still needs to be fixed. EMS-ESP32/src/devices/thermostat.cpp Line 335 in 8c4e5e5
|
Hi @Shmul1, if you don't mind I'd like to use some of your data as test data for EMS-ESP. I don't have any mixers, gateways or even a thermostat than can support more than one heating circuit! When you have time would you mind sending over some data which I can re-use? thanks |
Hello, sure, no problem. I'm the only one who can't make a video yet, I'm still on a business trip, otherwise I'll do what I can do remotely. |
Good afternoon.
My heating system has the following equipment:
I use 5 MM100 modules for heating, 1 MM100 module for hot water and 1 MM100 module for heating the pool.
If I control the system via the Thermostat CW400, then everything works fine for me.
Now I decided to connect the heating system to the Home Assistant via MQTT and I bought BBQKees Gateway E32. I can see all my equipment perfectly in the EMS-ESP web interface, but there is one problem.
Mixer with 1-4 are shown as h1-h4, 5 becomes wwc1 Mixer, Mixer 6 becomes wwc2, but the mixer 7 (id8 for hot water) is wwc1!!!
Thus in EMS-ESP got two mixer wwc1, MQTT and considers them as one and constantly reads the information for sensors from one to the other mixer!!!
Is it possible to rename the mixer so that they are displayed correctly via MQTT?
Is it possible, if there are more than 4 mixers in the system, then, for example, mixers 1-8 should be assigned the values h1-h8, and mixers 9 - wwc1, 10 - wwc2 (for CW800 users, for example)?
The text was updated successfully, but these errors were encountered: