forked from tbnobody/OpenDTU
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Prepare Release 2024.10.22 #1341
Merged
Merged
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
This is related to tbnobody#2235 and fixes tbnobody#2230
adds support for w5500 Ethernet chip, present on OpenDTU Fusion PoE Ethernet hat in particular.
I noticed that some useful ESP stats are missing on the MQTT broker, so this adds: - ESP temperature - ESP heap stats (size, free, minFree, maxAlloc)
When the BMS provides a discharge current limit, apply this limit in the DPL to the inverter power target when running from battery.
* implements UI to configure battery discharge limit * adds support for discharge limit to MQTT battery provider * add option to hide `issues` section from battery live view (for MQTT battery)
Allows to connect to SBS Unipower batteries using a CAN bus.
these were never adapted to our fork but merely copied form the upstream project.
* explicitly disable NRF24 when using CMT + W5500. fixes #1257. * explicitly disable CMT when using NRF + W5500 * added missing LED mapping for W5500 profiles
we need shell=True as on Windows, path resolution to find the yarn "exectuable" (a shell script) is only performed by cmd.exe, not by Python itself. as we are calling yarn with fixed arguments, using shell=True is fine.
it seems this action has not been run for nealry a year. it has only been run twice at all. it is not maintained, as it uses older Node.js in particular. also, it seems to be redundant to build.yml, which already builds firmware for every commit.
we need to set board_upload.flash_size = 8MB for the firmware to actually boot using the 8MB partition scheme.
shows whether or not known tasks are alive, and in particular shows how much of the respective stack is still available.
Due to a change in the Espressif Arduino core, the TX timeout for the HW CDC (used in the ESP32-S3, for example) must not be set to 0, as otherwise, an integer underflow occurs. Removing the TX timeout is not necessary anymore anyways, because it is now detected when CDC is not active, and attempts to write will return immediately until the host read something again. Only when the transmit buffer becomes full initially, the default timeout of just 100ms takes effect once. For USB CDC (used with the ESP32-S2, for example), the timeout is not relevant either.
shows whether or not known tasks are alive, and in particular shows how much of the respective stack is still available.
* Allow scaleValue() for 32bit values * Victron: Implement CAN message 0x360 This one-byte message is set to 0xff to request charging below a certain SoC threshold (10% in my tests). * Pytes: Add support for native CAN protocol The recently added PytesCanReceiver.cpp implements the Victron CAN protocol. This change additionally adds support for the native Pytes CAN protocol messages. Features only supported in Pytes protocol: - High-resolution state of charge / full and remaining mAh - Charge cycle counter - Balancing state Features only supported in Victron protocol: - FW version - Serial number Note that the only known way to select the native Pytes protocol is via the serial console (Cisco-compatible cables work): ``` login config setprt PYTES logout ``` to return to Victron protocol use: ``` login config setprt VICTRON logout ``` to return to DIP-switch based protocol setting: ``` login config setprt DIP logout ```
If the default type is an int, setting to a fractional value is not possible. Presumably this is because in `target.DischargeCurrentLimit = source["discharge_current_limit"] | BATTERY_DISCHARGE_CURRENT_LIMIT;` the JSON library uses the default to force the expected type and on type mismatch the default is used. As per https://arduinojson.org/v7/api/jsonvariant/or/: `defaultValue: the value to return if the JsonVariant is null or incompatible with the requested type.`
When the limit is valid, use the actual configured limit, not the bool value :)
a battery temperature value measured by a Victron smart battery sense and communicated to a connected Victron MPPT charge controller will now appear in the live view card.
…#1324) Enables horizontal scrolling for the batter views in order to not break the UI on small screens.
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.
Merges development into master.