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

Nilan Compact P (air/geo) GEO entities missing from the list. #160

Open
Hannes165 opened this issue Nov 30, 2024 · 35 comments
Open

Nilan Compact P (air/geo) GEO entities missing from the list. #160

Hannes165 opened this issue Nov 30, 2024 · 35 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@Hannes165
Copy link

Hi,

Added waveshare rs485/wifi adapter to the nilan compact P machine (replaced the original wifi gateway). Configured everything successfully and got the connection established.
In HA, my nilan shows up with 50 entities. Most of the GEO part (device 44)parameters seems to be missing. Any idea how to get them visible? Or what to check for troubleshooting?

@veista
Copy link
Owner

veista commented Dec 1, 2024

What specifically is missing? Can you share the requested information?

@Hannes165
Copy link
Author

Added the pictures of available entities and device info.
available after installation
device info after installation
AIR side seems to be all there. CO2 value didnt find.
GEO side. Brine (geo circuit) temp in out missing. Floor circuit in out missing. Compressor state and load missing. Pressure data missing. These should all be in modbus addresses 200-222. T14 is indicating the supply water temp, and it actually is this temp, but on the mashine itself, this temperature is marked T17. All setpoints missing.

Modbustester tool gives me all values if testing adresses regarding the registry from the nilan HACK files. This indicates that the connection details should be OK.

I can provide more info, i just do not know what specific info needed😉

@Hannes165
Copy link
Author

Hannes165 commented Dec 5, 2024 via email

@veista
Copy link
Owner

veista commented Dec 6, 2024

Turn debug logging on and reinitialize the integration. Give me the log.

@Hannes165
Copy link
Author

Hannes165 commented Dec 6, 2024 via email

@veista
Copy link
Owner

veista commented Dec 6, 2024

You need to reply on the website.

@Hannes165
Copy link
Author

@veista
Copy link
Owner

veista commented Dec 6, 2024

Could you please filter the log to only include nilan integration.

@Hannes165
Copy link
Author

this i do not know how to do. Im wondering why there is not only nilan activity, since i initiated the log from nilan integration page?

@veista
Copy link
Owner

veista commented Dec 6, 2024

  1. Enable debug logging for integration
  2. Clear logs
  3. Reload nilan integration
  4. Search custom_components.nilan in logs
  5. View raw log
  6. Enable condensed view

Screenshot_20241206_170724_Home Assistant

@Hannes165
Copy link
Author

@veista
Copy link
Owner

veista commented Dec 6, 2024

is that OK? home-assistant_2024-12-06T16-29-02.525Z.log

Does this look like the example I gave you?

@Hannes165
Copy link
Author

@Hannes165
Copy link
Author

settings/system/logs...
image
Hopefully this will help😉

@veista
Copy link
Owner

veista commented Dec 6, 2024

Yes. So you have a model with AIR or GEO installed? According to the integration logic you do not. This can be a bug in my code or some outlier in Nilan logic if you do have geothermal unit installed.

@veista
Copy link
Owner

veista commented Dec 6, 2024

Added the pictures of available entities and device info. available after installation device info after installation AIR side seems to be all there. CO2 value didnt find. GEO side. Brine (geo circuit) temp in out missing. Floor circuit in out missing. Compressor state and load missing. Pressure data missing. These should all be in modbus addresses 200-222. T14 is indicating the supply water temp, and it actually is this temp, but on the mashine itself, this temperature is marked T17. All setpoints missing.

Modbustester tool gives me all values if testing adresses regarding the registry from the nilan HACK files. This indicates that the connection details should be OK.

I can provide more info, i just do not know what specific info needed😉

Also you can't have both AIR and GEO installed, so you must be misunderstanding something here.

For CO2 to work you need to have CO2 sensor installed. If you do, this is an another bug.

@veista
Copy link
Owner

veista commented Dec 6, 2024

Take a picture of the device type plate and submit it with the issue.
If you have HMI350T - Touch screen HMI - installed on your device, take a picture of the device info page and submit it with the issue.
If you have CTS602 HMI, take a picture of "SHOW DATA" -> "TYPE" and submit it with the issue.

Can you provide these. Thanks!

@Hannes165
Copy link
Author

device Type plate:
WhatsApp Image 2024-12-06 at 20 17 03_6ea3d88b
CTS602 HMI/show data/unit information:
WhatsApp Image 2024-12-06 at 20 13 55_42399e7a
WhatsApp Image 2024-12-06 at 20 13 55_22d43ea5
CTS602 board:
WhatsApp Image 2024-12-06 at 20 15 52_8e50bb97

@veista
Copy link
Owner

veista commented Dec 6, 2024

Thanks. There is obviously a problem with the GEO recognition logic.

@Hannes165
Copy link
Author

I have installed the CO2 sensor and i can read it from HMI, also can read it with modbustester. The unit has two heat pumps inside, one for AIR, second for GEO.
HMI home screen:
WhatsApp Image 2024-12-06 at 20 13 55_cef89511
Show data has two units under it:
WhatsApp Image 2024-12-06 at 20 29 16_27284a33
Geo unit data:
WhatsApp Image 2024-12-06 at 20 30 44_050fb19d
WhatsApp Image 2024-12-06 at 20 30 44_11b4cd19
WhatsApp Image 2024-12-06 at 20 30 44_b42cc9e7
WhatsApp Image 2024-12-06 at 20 30 44_8f12687c
WhatsApp Image 2024-12-06 at 20 30 44_15903aae
This and the AIR side sensors are under modbus addresses 200-222.

@veista
Copy link
Owner

veista commented Dec 6, 2024

AIR is an external heatpump, so you misunderstand that functionality. It is a big unit installed outside.

@Hannes165
Copy link
Author

OK. In my case i was refering the air side heat pump inside my nilan device. It has Ventilation unit with heat pump on the extraction side. Heating supply AIR and DHW. Second part on the bottom of the device is GEO thermal heat pump primerily for floor heating, also available for DHW heating if system built acordingly.

@Hannes165
Copy link
Author

image
image

@veista
Copy link
Owner

veista commented Dec 6, 2024

OK. In my case i was refering the air side heat pump inside my nilan device. It has Ventilation unit with heat pump on the extraction side. Heating supply AIR and DHW. Second part on the bottom of the device is GEO thermal heat pump primerily for floor heating, also available for DHW heating if system built acordingly.

I see, that is just confusing! AIR is air-water heatpump and you are talking about the regular indoor heatpump. All of this should be implemented, you are probably the first user to have this device. If you can manually read the registers, we can debug this easily.

@Hannes165
Copy link
Author

I will add here the Show Data/Compact P as well:
WhatsApp Image 2024-12-06 at 20 56 52_563cfc2c
WhatsApp Image 2024-12-06 at 20 56 52_43e98831
WhatsApp Image 2024-12-06 at 20 56 52_80e0aa4c
WhatsApp Image 2024-12-06 at 20 56 53_7de161a2
WhatsApp Image 2024-12-06 at 20 56 53_073af635
WhatsApp Image 2024-12-06 at 20 56 53_0b365379

@veista
Copy link
Owner

veista commented Dec 6, 2024

I think I found the first issue. It will take some time to test and implement.

@Hannes165

This comment was marked as duplicate.

@veista
Copy link
Owner

veista commented Dec 6, 2024

Most issues should be resolved with latest release!

@veista veista added the bug Something isn't working label Dec 6, 2024
@Hannes165
Copy link
Author

update installed, HA restarted. It is impressive. Now there are two devices visible. Most of the sensors are on GEO side. For the Air unit compressor the state is running, but for the geo unit compressor there is no state indicated. Compressor voltage and capacity is visible but no state indication. Shall i try to list the sensors whitch should be on the AIR side of the nilan unit? Circulation pump is a mysterious thing. The geo unit has circulation pump for the Brine circuit, and this is Off right now. On top of that there is floor heating pump what is always running if in winter mode. And this is connected to one of the relays on the board. This can not be seen from the list. Otherwise the number of entities is now 116.

image

image

@Hannes165
Copy link
Author

There are some minor things like LED 1; 2. Display row 1; 2. These are not for device 44 i think. Also T2 is no longer visible. Supply air temp is visible, but with T7 on its name. On device 44 it is called Supply air T2.
Also the status of Brine circulation pump, unknown. And the status of the Floor heating circulation pump. Can i somhow assist you to get this added/fixed

@Hannes165
Copy link
Author

There are some minor things like LED 1; 2. Display row 1; 2. These are not for device 44 i think. Also T2 is no longer visible. Supply air temp is visible, but with T7 on its name. On device 44 it is called Supply air T2.
Also the status of Brine circulation pump, unknown. And the status of the Floor heating circulation pump. Can i somhow assist you to get this added/fixed.

Water setpoint is DHW heating setpiont for GEO pump, if enabled. Heating setpoint is the temperature the pump wants to supply to the heating (T17) and this is dynamically chande acording to the outdoor temperature temperature compensation curve.
Here are te GEO side sensors.
image

@Hannes165
Copy link
Author

I will leave tomorrow for some weeks. I can not give any feedback during that time. Many thanks and will revert later if anything comes up.

@veista
Copy link
Owner

veista commented Dec 7, 2024

There are some minor things like LED 1; 2. Display row 1; 2. These are not for device 44 i think. Also T2 is no longer visible. Supply air temp is visible, but with T7 on its name. On device 44 it is called Supply air T2.
Also the status of Brine circulation pump, unknown. And the status of the Floor heating circulation pump. Can i somhow assist you to get this added/fixed

Yes. Remove integration. Add integration.

@veista
Copy link
Owner

veista commented Dec 7, 2024

There are some minor things like LED 1; 2. Display row 1; 2. These are not for device 44 i think. Also T2 is no longer visible. Supply air temp is visible, but with T7 on its name. On device 44 it is called Supply air T2.
Also the status of Brine circulation pump, unknown. And the status of the Floor heating circulation pump. Can i somhow assist you to get this added/fixed.

Water setpoint is DHW heating setpiont for GEO pump, if enabled. Heating setpoint is the temperature the pump wants to supply to the heating (T17) and this is dynamically chande acording to the outdoor temperature temperature compensation curve.
Here are te GEO side sensors.
image

I will take a look. Most likely you will have to rename these yourself.

@veista
Copy link
Owner

veista commented Dec 7, 2024

I will take a look at wrongly named entities. This might not be fixable at integration level. They might be different only on your device. I can't rename only on your device.

If entities are missing, please reference manual and HMI panel. I will take a look into adding these in the future.

@veista veista added the enhancement New feature or request label Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants