tibber / com.tibber.athom

Tibber app for Athom Homey
17 stars 7 forks source link

Tibber 1.4.6 crashes a few seconds after starting up #5

Closed TheArcturian closed 3 years ago

TheArcturian commented 3 years ago

As soon as Tibber version 1.4.6 was automatically installed, it crashed a few seconds. Restarting the app or Homey does not help. Diagnostics should already be received plentiful by you. Please fix the bug. Thanks!

TheArcturian commented 3 years ago

Still crashes after upgrading to 1.4.7, but now it crashes when I create a "phase changed" flow and it gets triggered.

toini commented 3 years ago

We'll investigate this. To confirm: you are first seeing the currents per phase ok and the crash occurs after triggering the phase changed flow? I tried looking for a crash report from you but did not see one. Could you send it manually and share me the id?

toini commented 3 years ago

I noticed a bug in the push notification flow. It might not be related but there's anyhow a new test version https://homey.app/en-us/app/com.tibber/Tibber/test/. Could you install that, try the phase current flow and send diagnostics report and share its id?

TheArcturian commented 3 years ago

Yes, I will. Installing now. Getting back to you later.

TheArcturian commented 3 years ago

Seems to be working now so that is good. Thanks!

On a sidenote: When creating a flow and using the Pulse Device as trigger card, I can choose between "The power changed" and "Power changed". Seems like a double listing that needs a cleanup.

Also the "Tibber push notification" card doesnt send out the first text line. Only text line 2.

toini commented 3 years ago

This is the only trigger with the word power in pulse device configuration:

{ "id": "power_changed", "title": { "en": "Power changed" }, "tokens": [ { "name": "power", "type": "number", "title": { "en": "Power" } } ], "args": [ { "type": "device", "name": "device", "filter": "driver_id=pulse" } ] },

Could you try to remove and re-add the device so we can see if it gets cleaned up?

TheArcturian commented 3 years ago

I removed the Pulse device after installing 1.4.7 and had to update a bunch of flows so I'd rather not do that again with 1.4.8. Here is a screenshot of the list, however it is just a minor troublefree problem:

Skjermbilde

After testing a while today, it seems like the flow card "Current phase 3 changed" doesn't work. It never gets triggered even though I see in the Tibber app that it should. It does work when I click Test strangely enough. The duplicate flow with the card "Current phase 1 changed" works perfectly.

TheArcturian commented 3 years ago

Here is two screenshot of the 2 flows for phase 1 and 3:

Skjermbilde3 Skjermbilde2

In the Tibber app, the measurement of phase 1 is usually around 28. For phase 3 it is around 23. I set the trigger to 9 just to force it to trigger. Here you can see the current values in the web based flow tool:

Skjermbilde4

I also have tested to set the trigger value from 9 to 25 and turn on an oven to make it cross the threshold, but it still doesnt trigger and send the messages. Only when I press Test and set the value higher than the when clause.

lovethyresson commented 3 years ago

I've reproduced the same error here. @toini, let's look at this when you've got a chance.

toini commented 3 years ago

@TheArcturian the issue with "Tibber push notification" card not sending out first text line should now be fixed. Could you try it out?

We are still investigating the trigger issue for the currents.

TheArcturian commented 3 years ago

Ok, I'll let you know tomorrow.

TheArcturian commented 3 years ago

Moving away from Athom Homey to Home Assistant. Need a much better and responsive solution as a base. Hope to see the these triggers in the Home Assistant integration as well. :-)

lovethyresson commented 3 years ago

Do we support these already for HA @Danielhiversen ?

Danielhiversen commented 3 years ago

These Pulse data are available for automations in HA:

power
powerProduction
accumulatedProduction
accumulatedConsumption
accumulatedCost
currency
minPower
averagePower
maxPower
voltagePhase1
voltagePhase2
voltagePhase3
currentL1
currentL2
currentL3
lastMeterConsumption
lastMeterProduction
signalStrength
TheArcturian commented 3 years ago

Brilliant! :-)

toini commented 3 years ago

There's a new test version that solves the issue with L2 / L3 not getting triggered. You can install it here: https://homey.app/en-us/app/com.tibber/Tibber/test/

TheArcturian commented 3 years ago

Phase 3 now triggers as expected.