Age | Commit message (Collapse) | Author | Files | Lines |
|
Changes:
- Rework KUKSA.val "VAL" gRPC API implementation to separate it
from the vehicle model + notifier, and more easily allow using
it from other notifiers.
- Move volume handling from the vehicle model + notifier to the
audio set for clarity.
- Wire up the new VSS audio signals in the audio notifier. The
"rearFront" variable naming has been changed to "fade" in
several places to match expected terminology.
- Add a balance slider to the audio settings page.
- Change the min/max labels on the fade slider to be Text instead
of Icon's since we do not have the equivalent to use with the
balance slider, and text seems like it'd be what you would want
for any potential future internationalization.
- Rework configuration file to be usable from anywhere via a
RiverPod Provider instead of tied to the vehicle notifier code,
and shifted the background and hybrid animation flags to be handled
with it. This change removes the built-in asset with defaults in
favor of maintaining the defaults for the ICS environment in the
AppConfig and KuksaConfig classes, with a goal of avoiding the need
for using async methods in the config provider.
- Change some notifiers from using StateNotifier to the RiverPod
2.0 Notifier class for improved flexibility. The other notifiers
will be updated in future work.
- Added select's to several ref.watches in the new hybrid animation
code to avoid unnecessary repaints.
- Fix several spelling issues in method and parameter names
across the codebase.
Bug-AGL: SPEC-5001
Change-Id: Iefae417fa870405d659303497d96e519e6b6d1de
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
This commit changes central hybrid animation on dashboard screen.
For now we can set from env randomHybridAnimation variable.
If true animation should be switching randomly. If false just
setting proper hybrid mode based on speed and RPM value.
Also increase childLocks touch target
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I71f09c63bd1bfeda174f92feafc58019c23d07cb
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Changes:
- Fixed parsing of engine speed from databroker response to use
uint32 again to match VSS type.
- Shifted engine speed definitions in Vehicle class next to
vehicle speed since that seems a bit saner than having it down at
the bottom with the temperature values.
- Removed duplicate temperature state from temperature control
widget, and pushed the use of the providers from the HVAC widget.
The existing scheme was somewhat broken with respect to having the
values managed with RiverPod, e.g. new values from external updates
would not show up until switching away and back to the HVAC tab.
- Moved HVAC "SYNC" button state into the Vehicle state class to
manage it with RiverPod, and allow use from both the HVAC and
temperature widgets so that the expected behavior can actually be
implemented.
- Fixed brace usage in a couple of debugPrint statements.
Bug-AGL: SPEC-4999
Change-Id: I07ea49d20fedca47e6f9e54f45ad34296e0a873c
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
Changes:
- Fix sets of HVAC and volume control signals to update the
actuator target value instead of the current value. With the
KUKSA.val ecosystem, it is the actuator implementation / backend
that updates the current value after actuating.
- Add authorization token to subscribe request metadata if it is
present.
- Switch type used for engine speed to double to match VSS signal
definition. This fixes receiving updates, and avoids needing
to convert types as well.
- Added logic to push out the HVAC fan speed setting and handle
external updates.
- Remove unused core/constants/paths.dart file to avoid confusion
over VSS signals used and their variable naming.
Bug-AGL: SPEC-4999
Change-Id: Ifbf69af25e3c563e9c707a4145089e4242b89bcd
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
This also removes commented out code.
Original by Sabin Sajeevan <ssajeevan@ics.com>.
Bug-AGL: SPEC-4971
Change-Id: I31c3b3046f6e3bdd9cc641d403467eda11144f31
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I58c7f3281260fecc04515b8b7680955ab066c9eb
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I80ed2fb908488dce3eb88b2cdb361ea9802d9a6b
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: Ibbb322950b680a50629dfdc40ecd7bb3e481728d
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Original from Sabin Sajeevan <ssajeevan@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I7a961e57715fdbf8b05f54dfcf8e56159cadd068
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I95f23d4f475aa7db82b6d34f2fc62e9f10092a75
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Change-Id: Ia4100f300b7f8507564ef80a1fc56c2352cfd5ce
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Bug-AGL: SPEC-4971
Change-Id: I46b8567195f932953570136374f3308fbbb9fd52
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
The update of the ValClientHelper constructor arguments seems to
have been missed in a rebase of an internal change, update caller
to fix the build.
Bug-AGL: SPEC-4971
Change-Id: Ie6342bbf7bdfcd4d0945db078b5f002d2ebfccbb
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
Also:
- Remove some warnings.
- Improvements of hvac temp buttons.
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: Ic10e9ec5ec79f79f63d951469924325f29ea3fa7
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Fixes ICS' internal AGL-48,49,50.
Original from Sabin Sajeevan <ssajeevan@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I66c875551a69a1b53eee2d6e1d3fa725b20ff41b
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Remove some warnings.
Original from: Dominik Wawrzonek <dwawrzonek@ics.com>
Bug-AGL: SPEC-4971
Change-Id: I3693fa74f7995b4dcb016812403ed800702cb81c
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Rework Kuksa databroker configuration to also read a configuration
yaml file from /etc/xdg/AGL/ics-homescreen.yaml at runtime to allow
overriding the built-in default configuration. The latter is left
as is with the addition of an explicit disabling of TLS, to still
match the internal test setup at ICS. The AGL build will supply
a runtime configuration that enables TLS and an authorization token.
Additionally:
- the ValClientHelper constructor has been changed to take the
stub and authorization token as arguments to allow authorization
with the VAL API requests. The channel argument has been
removed since it is not used, only the stub is.
- the apparently unused vss_provider.dart source file has been
removed to avoid potential confusion.
Bug-AGL: SPEC-4969
Change-Id: Ic7b867eaf565ef22f2e70df0afe7b5272b604d04
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
Bug-AGL: SPEC-4969
Change-Id: I7af59c4df7b3697d01f31d174bc808b87413cb66
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
The application was failing to build due to the presence of the
"config" directory, which seems to conflict with the generation
of files with "flutter config" during builds with meta-flutter's
flutter-app bbclass. Rename the directory and update references
to it to allow building.
Bug-AGL: SPEC-4969
Change-Id: I5d36d8fb2c91f0a1aa6c4deb86a040424bc35ac5
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
|
|
Originally by david.poole <dpoole@ics.com>.
Bug-AGL: SPEC-4969
Change-Id: Ia22852ae639bd38ab77f74c58f8a763f4ef39e02
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|
|
Based on agldemo2024 on commit 2a5dc04d801134338150c3f6afc67eaa65599763
Disable device preview.
Disable Lottie animation.
The original commit was b3c493c340fcb4bb0a937692838fc830bec3e9ea
but I am just keeping this change, because the json did not really
needed to change. I think.
Signed-off-by: Lisandro Pérez Meyer <lpmeyer@ics.com>
|