Conversation
- Add SungrowBatteryConfig struct with nameplate capacity and module count - Support battery models SBR064 through SBR256 (2-8 modules) - Dynamically set battery model ID in 0x705 based on module count - Initialize module-dependent CAN frames in setup(): - 0x71A: cell type per module - 0x71B-E: production dates - 0x71F: serial numbers - 0x715-718: module voltage overview - Add web UI dropdown for battery model selection (Sungrow inverter only) - Conditionally transmit 0x71F frames based on configured module count
Previously only SBR096 and SBR128 were available in the web UI dropdown. This adds all supported Sungrow SBRXXX battery models that the inverter protocol already supports: SBR064, SBR160, SBR192, SBR224, and SBR256.
- Dynamically populate SOC frames (0x70F_05/06/07) based on module count - Add missing 0x71D and 0x71E production date frames to batch C - Make module 3 serial number transmission conditional (min 2 modules) - Unpopulated module slots transmit zeros as per protocol
…ltemps Volkswagen MEB: Fix negative reading of celltemps
- Change stored value from module count (2-8) to model index (0-6) - Rename get_config_for_modules() to get_config_for_model() - Use options_from_map() for cleaner web UI dropdown generation - Remove 7 individual INVBTYPE_SBRxxx handlers
Sync with main
Motor info A and B added to transmitted frames to keep DTCs for "Lost communication with Motor control unit" away.
for more information, see https://pre-commit.ci
…ed-phev-support Kia Hybrid: Improvement; Add autodetect between HEV and PHEV
sync with main
BECom Hardware - Initial implementation
…pa-current-direction Bugfix; Volvo SPA, Invert current direction
* Initial work on BMW i3 balancing * Initial work on BMW i3 balancing, pre-commit fixes * Initial work on BMW i3 balancing, pre-commit fixes2 * Initial work on BMW i3 balancing, pre-commit fixes3 * Initial work on BMW i3 balancing, fixes * Faster and more stable shutdown of battery and handle different battery sizes * Check if battery info is available before using it * Added button to end balance * Now green standby balancing status * Should get out of STANDBY again * Refetch battery info * odlg settings * Fix review comments * Revert "odlg settings" This reverts commit b53111d.
…attery-type Solax: Make battery type configurable again
Signed-off-by: madymax <darkmadymax@yahoo.com>
…rge-test-mode Precharge: Add autostart of precharge when running without battery
Signed-off-by: madymax <darkmadymax@yahoo.com>
for more information, see https://pre-commit.ci
…n485 Add display feature on Lilygo T-CAN485
…all_boards Compile the display on all boards
…lear-DTC Stellantis ECMP: Add DTC clear button
…name-deye-fixes Improvement: Rename Deye specific fix
…eedup-fixes Improvement: Make core loop faster
…more-info Pylon Battery: Ask battery for cellvoltages
Renault Kangoo: Correct PID polling
This merge integrates all upstream changes from Battery Emulator v10.2.dev into the ESP32-S3 port, including: - New battery support: Ford Mach-E, CMP Smart Car, Geely SEA, Growatt HV ARK, Tesla Legacy, Think - New VCU-CAN inverter protocol - ESP-NOW support - Display driver improvements - Watchdog and safety enhancements - MQTT and webserver updates - Multiple battery protocol improvements ESP-IDF 5.x compatibility fixes: - Fixed watchdog API (both ESP32 and ESP32-S3 now use config struct) - Fixed LEDC PWM API (both platforms use ledcAttachChannel) - Fixed MQTT config structure (nested structure for ESP-IDF 5.x) - Disabled new I2C display driver for ESP32-S3 (requires ESP-IDF 5.x i2c_master.h) - Updated PlatformIO espressif32 platform from 6.12.0 to 6.13.0 - Removed strict compiler warnings to work around raw string literal bug All builds verified: - esp32s3box: SUCCESS - lilygo_330: SUCCESS 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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
14 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.
Summary
This PR merges all upstream changes from Battery Emulator v10.2.dev into the ESP32-S3 port, bringing 761 commits of new features and improvements while maintaining full ESP32-S3 compatibility.
What's New from Upstream
ESP-IDF 5.x Compatibility Fixes
The upstream code had backwards assumptions about API versions. Both ESP32-S3 and LilyGo ESP32 environments use ESP-IDF 5.x, which requires:
ledcSetup/ledcAttachPintoledcAttachChanneli2c_master.hnot available in Arduino-ESP32)Build Verification
All builds tested and verified:
Files Changed
Testing
Both ESP32-S3 and LilyGo environments compile successfully with all new features intact.