Open
Conversation
Clarified instructions for determining resonant frequency and exporting data.
Owner
|
Hey Sammy, there appears to be a typo in the typo fix (CVS --> CSV). |
min SoC defined by Marstek is 12.0%. Due to premature rounding, the cutoff is already at 12.9%. This is a suggestion to take into account unrounded numbers.
Author
|
My apologies that I mix up typos with code improvements in the same pull request. This is actually my first commit in github... |
Author
|
Bob, je hebt een mooi stukje software geschreven! Zoals je zelf al zei: het project is nog maar jong. Maar volgens mij snel populair aan het worden.
Ik ben er steeds meer van overtuigd dat een klassieke PID regeling niet het beste is om mee te sturen. Zoiets is bekend uit de procestechniek (heb je daar toevallig ervaring mee?) om te sturen in een verder random proces.
Het probleem van een thuisnetwerk is dat die helemaal niet random is. Zo zit ik bijvoorbeeld met een Quooker die voorspelbaar is qua gedrag, en veel beter gewoon helemaal geskipt kan worden. Elke keer als HBC daarop aanslaat, is het te laat. Een ander ding is de integraal: als ik het goed begrijp blijft die gewoon intact als het teken omslaat. Volgens mij is het efficienter als de integraal laten resetten naar 0, omdat kennelijk het (meestal) kortstondige hoge verbruik is weggevallen. Voor het laadproces met zonnepanelen hoeft dat echter mogelijk niet, want die volgt meer een random proces (bij gedeeltelijke bewolking). Al dit soort effecten zijn m.i. best goed te programmeren. Ben je dat met me eens? Als je geinteresseerd bent kan natuurlijk best eea aanleveren.
Inmiddels snap ik de eerste beginselen van node-red en heb al een sensor gemaakt voor mijn baseline verbruik. En volgens mij kan je met node-red ook een Fourier transformatie doen, maar dat heb ik nog niet geprobeerd.
Groet,
Luc / Sammygo
On Friday, 6 February 2026 at 20:51:02 CET, B. Reijnders ***@***.***> wrote:
gitcodebob left a comment (gitcodebob/marstek-venus-rs485-node-red#83)
Hey Sammy, there appears to be a typo in the typo fix (CVS --> CSV).
Could you update the documentation and push to this PR again?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Added recorder configuration to exclude specific entities.
Refactor load distribution function in JSON file
sammyjo468
commented
Feb 10, 2026
Author
sammyjo468
left a comment
sammyjo468
left a comment
There was a problem hiding this comment.
this commit (part on JSON file) has been superseded by a later commit on the same topic.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
proportinal -> proportional