Skip to content
This repository has been archived by the owner on Nov 18, 2020. It is now read-only.

Local improvements #33

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Local improvements #33

wants to merge 3 commits into from

Commits on Dec 16, 2019

  1. Changes to build using VSCode/PlatformIO

    @jennytoo
    jennytoo committed Dec 16, 2019
    Configuration menu
    Copy the full SHA
    3605f1d View commit details
    Browse the repository at this point in the history

  2. Expose Name variable to rename device

    @jennytoo
    jennytoo committed Dec 16, 2019
    Configuration menu
    Copy the full SHA
    ed08f37 View commit details
    Browse the repository at this point in the history

  3. : Set currentMode on start, fix led flicker 

    * LED update is based on the currentMode variable, not the Mode
variable. On initial startup getConfig sets Mode, but currentMode is
left unset. As this isn't a valid mode the refresh loop sets both values
to the default Colour, requiring the config be rerear. FIX: Set
currentMode when initializing the LEDs.
* More complex modes make the LEDs flicker; instead of running the
render code every update add a toggle to alternate between render or
show giving other code a chance to run between slow renders and
strip refresh. Most likely being caused by wifi processing/buffering and
this fixes it on my devices.
    @jennytoo
    jennytoo committed Dec 16, 2019
    Configuration menu
    Copy the full SHA
    1cac87c View commit details
    Browse the repository at this point in the history