-
-
Notifications
You must be signed in to change notification settings - Fork 784
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
chore: make decorate dependency-aware #15434
Conversation
@andig i am currently travelling. happy to help with this next week. |
@andig Die Tests schlagen fehl, weil der Heating-Use-Case damit nicht mehr korrekt funktioniert. Es wird % anstatt Celsius in der UI angezeigt. Test: https://github.com/evcc-io/evcc/blob/master/tests/heating.spec.js |
Ohja, böser Fehler und super schwer zu finden. Danke! |
Sehr schön, aber brauchen wir bei den Chargern überhaupt die |
Good point. Raus damit und dann rein damit :) |
Reduce combinatorial complexity by eliminating invalid combinations: https://github.com/evcc-io/evcc/pull/15434/files#diff-e149a97460e875de78b30ce55504f72e6e43b60ec9beac533c8b9cfef00cf828R32-R36.
On Mac Arm64, recreating assets reduces binary size from 95MB to 90MB.Binary size reduction is offset by adding optional values to charger.Refs and improves #14546: custom chargers can now directly implement phase
voltages
andcurrents
.Refs https://stackoverflow.com/questions/78881749/how-to-get-the-static-reflect-type-of-an-interface.