-
-
Notifications
You must be signed in to change notification settings - Fork 107
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
ahs / am200: web-api needs corrections #857
Comments
Please ask the user to participate here and post the system info. |
Hi, i am here and ready to test and give feedback. I think it would be the right way to implement a separate device class for the am200 module - as for mixers (MM100 and so on) or solar modules (SM100...). As i can see the implementation that has been done already also happened on user request and i really appreciate all the work you put into this. Great project! |
Yes, it's better to create a own device class. The boiler have a lot of entities making problems with buffer sizes. Boilers connected to a cascade module will still go to boiler class with tag HS1.., there are only a few values and the boilers keep there product-id, only changes device-id to 70... I'll make a pr. @proddy please check if you agree that this is the right way. |
@MichaelDvP yes, good to split off into its separate device class. Let me know how I can help |
@MichaelDvP what will be the device name within device-list of http://ems-esp/api/system ? Is it "heatsource" ? |
Hi, installed that firmware and it is working! From the log:
I can also confirm that the web API now returns details for the entities of the heatsource. http://ems-esp/api/heatsource/altflowtemp
So i would say it looks good ... @tp1de In KM200 mode my personal preference would be that the AM200 values would come up as hs2 under heatSources. Here is what http://ems-esp/api/system returns
For the EMS-ESP Dashboard / Devices & Sensors i would suggest that the Heatsource also gets the 'Boiler' icon and that it be named 'AM200 Alternative Heatsource'. What else can we check? I think not all entities are identified already, is that correct? Great work! Thanks... |
@tglynx |
@tp1de |
Ok, but i shorten the name to "AM200", all devices are nemd only RC300, i700, etc. the extrawas only because it wassorted as boiler but isn't one. I've looked at #573, where we added the AM200. @OlliHein, please also check the changes in https://github.com/MichaelDvP/EMS-ESP32/releases If you are using homeassistant there will be the entiies changed. Sorry, but the class change can't be done without HA changings. @tp1de If using a cascade module the boiler will have nests with boiler/hs1/ .. boiler/hs16/, make sure that will not conflict. |
good point @MichaelDvP ... I was already thinking about it. How can I identify the nested boilers and how many there are? Or might it be easier to name the ahs module just "hsa?". @tglynx what about "hsa" instead of fixed "hs2"? |
sure, i think it is just a matter of preference as the original KM100/KM200 gateways do not support the AM200 module. |
Mainly the Cascade MC400 is a single boiler to the bus with all entities. The bus supports only one boiler. For the connected boilers there is a new ems-bus for each and the MC400 is thermostat for these busses. Only a few telegrams are forwarded in a kind of NAT (network translation), mapping the real boilers to device-id 0x70, 0x71, ... Example: We have in database:
and someone connects one MC400 and four GB192/20kW
The hs1..hs4 give only these 5 stati and can not be written directly, it's just info. All commands go to MC400. |
@proddy Is this way ok? |
@MichaelDvP |
@MichaelDvP I'll go with whatever you think is the right way forward is. I haven't looked at your new code yet, but is the heatsource data being pulled into the boiler_data MQTT topic too? |
Just to mention:
Such a system with multiple / hybrid heat sources will have global (system-wide) fields/values as well as entries per heat source. |
That's the point, for mqtt we can go to single topics I think it's better to move. Mixers also have only tagged entities, than we have heatsources always tagged with ahs1, hs1, hs2, etc. I'll stick to ahs1, so we can more if there are any. @tp1de The hybrid system HM200 is listed as heatpump. Mixers could control hc1-8 or wwc 1-10, Poolmodule (PM100) or zonemodule MZ100. For RC300 some known dhw2 entities are already mapped, but nobody seems to use or miss it, so we don't know the telegrams/offsets. |
@MichaelDvP I wonder if I understand correctly: Heatsources are boilers, alternative heatsources (am200), heat pumps with or without hybrid manager (hm200): ... or in other words: |
I do like the generic term "heatsource". The word 'boiler' was back when I created the first version of this project in 2017. If we're going to do any massive renaming, now is time. And I like big disruptive changes because it's fun and annoying! |
Installed latest build. api/system:
api/heatsource:
api/heatsource/ahs1:
from the logs (querying the API with the ioBroker Adapter):
i like the approach with ahs1 as this does not conflict with hs1...hs2...
Yes ... at the same time :) |
try again. |
/api/heatsource returns
/api/heatsource/ahs1 returns
/api/heatsource/ahs1/altflowtemp (as an example) returns
looks good for the ems-esp part i would say... @tp1de will need to adjust the ioBroker adapter to support the heatsource structure in ioBroker now From my point of view it would be heatsource/ahs1/...entities... in native ems-esp mode and heatSources/ahs1/...entities... in KM200 mode. |
@tp1de can you check this again against the latest dev build and either comment or close it. thanks |
Everything works from my point of view ... I adjusted my ioBroker adapter to new version as well. |
Hello everyone, not sure if this is the right place to comment, but I also have an AM200 in my heating system and use a flue gas sensor on TF1 to measure the exhaust gas temperatures from my wood stove with a water jacket. My plan is when the temperatures fall below a certain value I trigger a notification through HA and know to throw some wood back on the fireplace without having to check every 30 mins XD. I do not see the flue gas temperature reading in the EMS ESP dashboard but do on my RC310. Anyone have the same behavior/use case? |
Hi, i am almost sure the exaust gas temperature is not yet implemented. You should really create a new issue for this (enhancement). As a preparation, could you please post a photo of this information on the RC310 (as i do not have that sensor / to get an idea where the RC310 displays that info)... also please do a telnet session to the ems-esp and type 'watch 60' ... let the output run for a while when having a fire burning. i assume the value will be in the amTemperatures message ...
If possible send some amTemperatures messages along with the temperature that is displayed on the RC310 at the same time the message has been received. Michael |
Hi Michael, thanks for the super quick response! I have attached a picture, however, the PT1000 isn't yet connected to the exhaust pipe, hence the low temperature reading as I'm still experimenting :) Tomorrow I should get around to installing the sensor in the exhaust and will add the telnet output when the fire is burning and I am picking up a feasible temperature reading from the sensor. So far when I run watch 60 I get the following from telnet 000+01:41:44.832 N 8: [emsesp] heatsource(0x60) -B-> All(0x00), AmTemperatures(0x054D), data: 80 00 80 00 80 00 02 BB 02 6E 02 6C 80 00 02 44 00 00 80 00 01 would a screenshot from my EMS ESP dashboard also help? |
okay, nice... you could already experiment with your current config by warming the PT1000 with your hands and see if you can identify the changing values in the AmTemperatures message (still assuming it is in that message) ... i also do not have a return temperature sensor but that value is implemented as far as i can see (so you can see AHS return temp in the dashboard, right)? what would also be helpflull ... please copy some AmTemperature messages for later analysis. It should be easy to identify if you have much larger values after tomorrow... |
Yea, a lot of Temperature readouts are actually already supported in the dashboard which is really great! It seems I'm quite lucky that support was recently added and so much dev work has been taking place to better the AM200 support 👌 I was experimenting with a heat gun and running upstairs to the RC310 controller and then back down to the AM200 in the basement and it is definitely picking up and spitting out feasible information! And seeing as it at least works in the EMS+ I will wait till my temperature resistant PT1000 arrives tomorrow and I get around to properly installing the sensor to the exhaust pipe to ping telnet. I also already created an enhancement here #906 as per your recommendation, so if you prefer to continue the conversation there I'm game 🙌 |
I got a GH issue in respect to my ioBroker adapter (tp1de/ioBroker.ems-esp#12) opened by @tglynx.
The ioBroker Adapter is polling ems-esp by using web-api calls.
I could find the following issues which need corrections:
From my point of view I recommend to adjust.
The text was updated successfully, but these errors were encountered: