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.

Sign up for GitHub

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

Jump to bottom

v0.5.65 tickIVCommunication trigger #534

Closed
Gerri1 opened this issue Dec 30, 2022 · 7 comments
Closed

v0.5.65 tickIVCommunication trigger #534

Gerri1 opened this issue Dec 30, 2022 · 7 comments
Assignees
@lumapu
Labels
bug Something isn't working fixed dev fixed

Comments

@Gerri1
Copy link

Gerri1 commented Dec 30, 2022

NTP timeserver unreachable und nrf24l01+ funkt fleißig weiter obwohl disable night communication! 😮
@lumapu
Copy link
Owner

lumapu commented Dec 30, 2022

Ihr seit ja schneller als das Licht - das korrigierte Release baut seit 6 Minuten

@knickohr
Copy link

Guckst Du 66 😅

@Gerri1
Copy link
Author

Gerri1 commented Dec 30, 2022

😪

@knickohr
Copy link

Es ist ein Junge, ähhh Stable 🥳

@Gerri1
Copy link
Author

Gerri1 commented Dec 30, 2022

Hurraaaa, ... da ist es ja!! 👍😊

@lumapu lumapu closed this as completed Dec 30, 2022
@beegee3
Copy link
Contributor

beegee3 commented Dec 31, 2022

@knickohr @Gerri1 ihr seid ja hammer: live dabei 🙇‍♂️

nrf24l01+ funkt fleißig weiter obwohl disable night communication!

@lumapu: in tickIVCommunication passt der Trigger nicht
Abhilfe schafft nxtTrig = mSunset + mConfig->sun.offsetSec + 1;
(alternativ: if (mTimestamp >= (mSunset + mConfig->sun.offsetSec)) { )

@knickohr
Copy link

@beegee3

Wer was will muß auch was dafür tun. Da ich zwar des klingonisch mächtig bin aber nicht dieser kryptischen Programmiersprache kann ich wenigsten so meinen Beitrag dazu leisten und jede Menge Issues auf machen 😂

Wir müssen mal Gagh essen gehen 😉

Qapla‘

@lumapu lumapu reopened this Dec 31, 2022
lumapu added a commit that referenced this issue Jan 8, 2023
@lumapu
corrected MQTT comm_disabled #529
d8e255d 
fix Prometheus and JSON endpoints (`config_override.h`) #561
publish MQTT with fixed interval even if inverter is not available #542
added JSON settings upload. NOTE: settings JSON download changed, so only settings should be uploaded starting from version `0.5.70` #551
MQTT topic and inverter name have more allowed characters: `[A-Za-z0-9./#$%&=+_-]+`, thx: @mo Demman
improved potential issue with `checkTicker`, thx @cbscpe
MQTT option for reset values on midnight / not avail / communication stop #539
small fix in `tickIVCommunication` #534
add `YieldTotal` correction, eg. to have the option to zero at year start #512
@lumapu lumapu self-assigned this Jan 8, 2023
@lumapu lumapu added the fixed dev fixed label Jan 8, 2023
@stefan123t stefan123t added the bug Something isn't working label Jan 12, 2023
@stefan123t stefan123t changed the title v0.5.65 v0.5.65 tickIVCommunication trigger Jan 12, 2023
modem-man-gmx pushed a commit to modem-man-gmx/ahoy that referenced this issue Jan 15, 2023
@lumapu @modem-man-gmx
corrected MQTT comm_disabled lumapu#529
6e41a83 
fix Prometheus and JSON endpoints (`config_override.h`) lumapu#561
publish MQTT with fixed interval even if inverter is not available lumapu#542
added JSON settings upload. NOTE: settings JSON download changed, so only settings should be uploaded starting from version `0.5.70` lumapu#551
MQTT topic and inverter name have more allowed characters: `[A-Za-z0-9./#$%&=+_-]+`, thx: @mo Demman
improved potential issue with `checkTicker`, thx @cbscpe
MQTT option for reset values on midnight / not avail / communication stop lumapu#539
small fix in `tickIVCommunication` lumapu#534
add `YieldTotal` correction, eg. to have the option to zero at year start lumapu#512
@lumapu lumapu closed this as completed Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lumapu lumapu

Labels
bug Something isn't working fixed dev fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@stefan123t @lumapu @knickohr @Gerri1 @beegee3