forked from tbnobody/OpenDTU
-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
Merge upstream v25.1.14 #1534
Merged
schlimmchen
merged 44 commits into
development
from
andreasboehm/merge-upstream-v25.1.14
Jan 14, 2025
Merged
Merge upstream v25.1.14 #1534
schlimmchen
merged 44 commits into
development
from
andreasboehm/merge-upstream-v25.1.14
Jan 14, 2025
Conversation
This file contains 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
add Sum of DC power of all enabled inverters to Homeassistant MQTT autodiscovery
if a page uses the reload button, it had only 1 column of space, and only if the viewport was at least "sm". this is not the case for typical smartphones, in which case the reload button would appear on its own row instead of to the right. we now limit the heading to 10 columns if and only if the reload button is to be used, otherwise the heading uses all 12 columns, regardless of the viewport size. the reload buton uses two columns -- if it is displayed at all. the font size of the icon is increased slightly. as the font size of h1 headings changes with the viewport size, we need to center both the heading and the button vertically.
if variables are set with 'const foo = {} as Inverter', then 'foo.serial' will be undefined, causing warnings and errors when using InputSerial.
* increase the spacing between the icon and the text. * put the text into its own box so it does not flow around the icon. * vertically center the icon to account for multiple lines of text. * vertically center the text to account for a single line of text. closes #1441.
For some reasons this leads to WDT resets on this kind of module. This is just a workaround until another solution is found.
Goal of this change is to prevent a overflow in the command queue by flooding it with MQTT commands and therefor also prevent the reading of the inverter data. To achieve this it is now possible to specify a insert type for each queue element.
…n any more commands
…ple inverters are available)
The third parameter should be a optional channel name and not a scan method. There exists a separate method for the scan method.
The detection of 'Power Distribution Logic' is based on the firmware version for specific models and is needed to disable any means of overscaling, as it simply does not work when 'Power Distrbution Logic' is available. Based on the code from @AndreasBoehm
4584b4f
to
65738b7
Compare
db4c098
to
13904a4
Compare
schlimmchen
approved these changes
Jan 14, 2025
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.