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

Any other debugging options? #14

Open
Uriziel01 opened this issue Jan 14, 2021 · 23 comments
Open

Any other debugging options? #14

Uriziel01 opened this issue Jan 14, 2021 · 23 comments

Comments

@Uriziel01
Copy link

Uriziel01 commented Jan 14, 2021

Hi,
I'm facing an issue that my HomeAssistant mqtt broker gets the messages from CZ-TAW1 but HAAutoDiscovery does not work properly, only value messages are posted to mqtt, no configuration messages at all. I also cannot do any "write" commands, I only get values from the board, actions like below does nothing.

image
image

Are there any additional ways except soldering serial port wires to the board to see what is happening? Maybe there should be a toggleable (most propably another configuration field) debug mqtt topic that posts console output?

My config:

Readonly=false
Device="/dev/ttyUSB0"
Loghex=true
ReadInterval=5
Aquarea2mqttCompatible=false
Aquarea2mqttPumpID="[REMOVED]"
Mqtt_topic_base="panasonic_heat_pump/main"
Mqtt_set_base="panasonic_heat_pump/commands"
ForceRefreshTime=300
MqttServer="192.168.[REMOVED]"
MqttPort="1883"
MqttLogin="[REMOVED]"
MqttPass="[REMOVED]"
SleepAfterCommand=1
MqttClientID="GoHeishaMon-pub"
MqttKeepalive=60
EnableCommand=true
HAAutoDiscover=true

P.S - Dzięki za Twoją ciężką pracę, super projekt!

@Uriziel01
Copy link
Author

Uriziel01 commented Jan 14, 2021

Maybe even something simple and basic like:
(and then we can try to wrap all most important logs in log_message wrapper)

func log_message(a string, mclient mqtt.Client) {
	fmt.Println(a)
	if config.Logmqtt == true {
                mclient.Publish(fmt.Sprintf("%s/mqttlog", config.Mqtt_set_base), byte(0), false, a)
	}
}

@lsochanowski
Copy link
Owner

Hi,

In our taw image there is SSH you can login to taw and run goheisha in interactive mode to watch detailed log ,also there is topic to execute commands on taw by mqtt Messages

Ha autodiscover is not finished yet ;/ and it's expected behavior you need to set topics manualy

There is two thing curently ongoing in project autodiscover and optionboard implementation

@Uriziel01
Copy link
Author

The mqtt OSCommand topic is also not working for me sadly so I'm unable to enable ssh that way :/ Any precompiled image with SSH enabled available?

I could help with HA AutoDiscovery (also want to have it working on my instance) if you are opened for PRs?

@lsochanowski
Copy link
Owner

PR's are welcome ! I'am stuck with that because i have to decide what type of entity is best for a write command (mqtt thermostat or input etc ) and because of this i stoped working with that feature ;)

@lsochanowski
Copy link
Owner

Every image is with SSH

@lsochanowski
Copy link
Owner

Oscommand must work can you give me some more info ?

@MiG-41
Copy link
Collaborator

MiG-41 commented Jan 14, 2021

SET functions are implemented only up to SET13 , so SetFloorHeatDelta not working (yet...)

@MiG-41
Copy link
Collaborator

MiG-41 commented Jan 14, 2021

Are there any additional ways except soldering serial port wires to the board to see what is happening?

Also soltering USB serial conwerter not help ,as i remember , since after starting GoHeishaMon serial console is blocking ( probabl it is some issue with serial speed , this one works with 115200 , GoHeishaMon change it to 9600 ( just like communication with HP) ,and no more logging...
But ssh is the best way.

@MiG-41
Copy link
Collaborator

MiG-41 commented Jan 14, 2021

And finally for Home Assistant there is ready some example on HeishaMon repository , and waiting for working HAAutoDiscovery you can always use: https://github.com/Egyras/HeishaMon/tree/master/Integrations/Home%20Assistant

@lsochanowski
Copy link
Owner

mqtt
i've checked and command work like designed ;)

@Uriziel01
Copy link
Author

Uriziel01 commented Jan 14, 2021

I've tried that and the result is sadly different :/ I'm sure this is some really dumb misteake on my configuration(?) side, but just cannot pinpoint what it is.

image

image

@Uriziel01
Copy link
Author

Uriziel01 commented Jan 14, 2021

It's propably not relevant at all but I'm not using WiFi, connecting only using ethernet (my wifi signal is very weak in the pump room)

@Uriziel01
Copy link
Author

@MiG-41 Thanks for the HASS yaml, it's way better that way than defining all of them on my own, had no idea there is such file. Tried that but it is really tedious 😋

@Uriziel01
Copy link
Author

You guys are right regarding the SET13 thing, tried holiday mode that is supported and it works, it enabled it on the pump. Still no luck with OSCommand

@lsochanowski
Copy link
Owner

ok what command you are listen to ? ;)

@lsochanowski
Copy link
Owner

ok i've seen you'r problem ;) i think so ;)

in you first post you give your config and you have

Mqtt_set_base="panasonic_heat_pump/commands"

then. OSCommand shoult'd be
panasonic_heat_pump/commands/OSCommand
and listen for out
panasonic_heat_pump/commands/OSCommand/out

it's because :
c.Subscribe(config.Mqtt_set_base+"/OSCommand", 2, HandleOSCommand)

@Uriziel01
Copy link
Author

Uriziel01 commented Jan 14, 2021

@lsochanowski Jesus... you are so right, panasonic_heat_pump/commands/OSCommand did the trick 👍 Now it's working as intended 🙇

@MiG-41
Copy link
Collaborator

MiG-41 commented Jan 15, 2021

So SSH is not working at your case ? Did you tried already Firstboot method described in read.me ? If not , can you log the command "uci show" via OSCommand/out and pass it ? ( there can be wifi creditials to remove)

@Uriziel01
Copy link
Author

@MiG-41 The hint from @lsochanowski resolved my issue with SSH ✌️ I've managed to compile my own modified taw2mqtt.go and I hope I'll be able to address some issues with PRs soon.

@lsochanowski
Copy link
Owner

Did you manage to enable SSH by mqtt can you provide some more info because we want to make sure it's covered by readme and our test taw are modified we need some Feedback from someone who do it on ,,clear" taw like you ;)

@Uriziel01
Copy link
Author

Sure, I'll try to also improve README content to account for my issue. Hope will have some time to do so in the upcomming weekend.

@MiG-41
Copy link
Collaborator

MiG-41 commented Jan 15, 2021

@Uriziel01 ,but you can also tell us here , what have to done additionally ?
Aslo what are your's modifications ? Maybe it can be siutable for more people :)

@MiG-41
Copy link
Collaborator

MiG-41 commented Feb 6, 2021

@Uriziel01 any news ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants