Merge branch 'dev' of github.com:blazoncek/WLED into dev

CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ### Builds after release 0.12.0
 
+#### Build 2107100
+
+-   Version bump to 0.13.0-b2 "Toki"
+-   Accept hex color strings in individual LED API
+-   Fixed transition property not applying unless power/bri/color changed next
+-   Moved transition field below segments (temporarily)
+-   Reduced unneeded websockets pushes
+
 #### Build 2107091
 
 -   Fixed presets using wrong call mode (e.g. causing buttons to send UDP under direct change type)

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "wled",
-  "version": "0.13.0-bl1",
+  "version": "0.13.0-bl2",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "wled",
-  "version": "0.13.0-bl1",
+  "version": "0.13.0-bl2",
   "description": "Tools for WLED project",
   "main": "tools/cdata.js",
   "directories": {

pio-scripts/output_bins.py

@@ -32,7 +32,7 @@ def bin_rename_copy(source, target, env):
 
     release_name = _get_cpp_define_value(env, "WLED_RELEASE_NAME")
 
-    if release_name and os.getenv("WLED_RELEASE"):
+    if release_name:
         _create_dirs(["release"])
         version = _get_cpp_define_value(env, "WLED_VERSION")
         release_file = "{}release{}WLED_{}_{}.bin".format(OUTPUT_DIR, os.path.sep, version, release_name)

wled00/cfg.cpp

@@ -263,7 +263,7 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
   JsonObject if_sync_recv = if_sync["recv"];
   CJSON(receiveNotificationBrightness, if_sync_recv["bri"]);
   CJSON(receiveNotificationColor, if_sync_recv["col"]);
-  CJSON(receiveNotificationEffects, if_sync_recv[F("fx")]);
+  CJSON(receiveNotificationEffects, if_sync_recv["fx"]);
   //! following line might be a problem if called after boot
   receiveNotifications = (receiveNotificationBrightness || receiveNotificationColor || receiveNotificationEffects);
 

wled00/data/index.css

@@ -643,6 +643,12 @@ input[type=range]::-moz-range-thumb {
 	-webkit-backface-visibility: hidden;
 	-webkit-transform:translate3d(0,0,0);
 }
+#tt {
+  text-align: center;
+}
+.cl {
+	width: 42px;
+}
 .sel-pl {
 	width: 165px;
 	background-position: 141px 16px;

wled00/data/index.htm

@@ -172,6 +172,7 @@
 		<div id="segutil2">
 			<button class="btn btn-s" id="rsbtn" onclick="rSegs()">Reset segments</button>
 		</div>
+		<p>Transition: <input id="tt" class="noslide" type="number" min="0" max="65.5" step="0.1" value="0.7">s</p>
 	</div>
 
 	<div id="Favorites" class="tabcontent">
@@ -181,8 +182,7 @@
 		</div>
 		<div id="pcont">
 			Loading...
-		</div><br>
-		Transition <input id="tt" class="noslide" type="number" min="0" max="65.5" step="0.1" value="0.7">s
+		</div>
 	</div>
 </div>
 

wled00/data/index.js

@@ -1200,6 +1200,11 @@ function requestJson(command, rinfo = true)
 	{
 		command.v = true; // force complete /json/si API response
 		command.time = Math.floor(Date.now() / 1000);
+		var t = d.getElementById('tt');
+		if (t.validity.valid) {
+			var tn = parseInt(t.value*10);
+			if (tn != tr) command.transition = tn;
+		}
 		req = JSON.stringify(command);
 		if (req.length > 1000) useWs = false; //do not send very long requests over websocket
 	}
@@ -1242,7 +1247,6 @@ function togglePower()
 {
 	isOn = !isOn;
 	var obj = {"on": isOn};
-	obj.transition = parseInt(gId('tt').value*10);
 	requestJson(obj, false);
 }
 
@@ -1639,7 +1643,6 @@ function setPalette(paletteId = null)
 function setBri()
 {
 	var obj = {"bri": parseInt(gId('sliderBri').value)};
-	obj.transition = parseInt(gId('tt').value*10);
 	requestJson(obj, false);
 }
 
@@ -1855,7 +1858,6 @@ function setColor(sr)
 	}
 	updateHex();
 	updateRgb();
-	obj.transition = parseInt(gId('tt').value*10);
 	requestJson(obj, false);
 }
 

wled00/fcn_declare.h

@@ -179,7 +179,7 @@ void _drawOverlayCronixie();
 //playlist.cpp
 void shufflePlaylist();
 void unloadPlaylist();
-void loadPlaylist(JsonObject playlistObject, byte presetId = 0);
+int16_t loadPlaylist(JsonObject playlistObject, byte presetId = 0);
 void handlePlaylist();
 
 //presets.cpp

wled00/html_other.h

@@ -42,7 +42,7 @@ function B(){window.history.back()}function U(){document.getElementById("uf").st
 .bt{background:#333;color:#fff;font-family:Verdana,sans-serif;border:.3ch solid #333;display:inline-block;font-size:20px;margin:8px;margin-top:12px}input[type=file]{font-size:16px}body{font-family:Verdana,sans-serif;text-align:center;background:#222;color:#fff;line-height:200%}#msg{display:none}
 </style></head><body><h2>WLED Software Update</h2><form method="POST" 
 action="/update" id="uf" enctype="multipart/form-data" onsubmit="U()">
-Installed version: 0.13.0-bl1<br>Download the latest binary: <a 
+Installed version: 0.13.0-bl2<br>Download the latest binary: <a 
 href="https://github.com/Aircoookie/WLED/releases" target="_blank"><img 
 src="https://img.shields.io/github/release/Aircoookie/WLED.svg?style=flat-square">
 </a><br><input type="file" class="bt" name="update" required><br><input 

wled00/html_settings.h

@@ -395,7 +395,7 @@ HTTP traffic is unencrypted. An attacker in the same network can intercept form
 <h3>Software Update</h3><button type="button" onclick="U()">Manual OTA Update
 </button><br>Enable ArduinoOTA: <input type="checkbox" name="AO"><br><h3>About
 </h3><a href="https://github.com/Aircoookie/WLED/" target="_blank">WLED</a>
- version 0.13.0-bl1<br><br><a 
+ version 0.13.0-bl2<br><br><a 
 href="https://github.com/Aircoookie/WLED/wiki/Contributors-and-credits" 
 target="_blank">Contributors, dependencies and special thanks</a><br>
 A huge thank you to everyone who helped me create WLED!<br><br>

wled00/json.cpp

@@ -143,13 +143,13 @@ void deserializeSegment(JsonObject elem, byte it, byte presetId)
   //temporary, strip object gets updated via colorUpdated()
   if (id == strip.getMainSegmentId()) {
     byte effectPrev = effectCurrent;
-    effectCurrent = elem[F("fx")] | effectCurrent;
+    effectCurrent = elem["fx"] | effectCurrent;
     if (!presetId && effectCurrent != effectPrev) unloadPlaylist(); //stop playlist if active and FX changed manually
     effectSpeed = elem[F("sx")] | effectSpeed;
     effectIntensity = elem[F("ix")] | effectIntensity;
     effectPalette = elem["pal"] | effectPalette;
   } else { //permanent
-    byte fx = elem[F("fx")] | seg.mode;
+    byte fx = elem["fx"] | seg.mode;
     if (fx != seg.mode && fx < strip.getModeCount()) {
       strip.setMode(id, fx);
       if (!presetId) unloadPlaylist(); //stop playlist if active and FX changed manually
@@ -181,22 +181,19 @@ void deserializeSegment(JsonObject elem, byte it, byte presetId)
           stop = iarr[i];
           set = 2;
         }
-      } else {
-        JsonArray icol = iarr[i];
-        if (icol.isNull()) break;
-
-        byte sz = icol.size();
-        if (sz == 0 || sz > 4) break;
-
+      } else { //color
         int rgbw[] = {0,0,0,0};
-        copyArray(icol, rgbw);
-
-        if (set < 2) stop = start + 1;
-        for (uint16_t i = start; i < stop; i++) {
-          strip.setPixelColor(i, rgbw[0], rgbw[1], rgbw[2], rgbw[3]);
+        JsonArray icol = iarr[i];
+        if (!icol.isNull()) { //array, e.g. [255,0,0]
+          byte sz = icol.size();
+          if (sz > 0 || sz < 5) copyArray(icol, rgbw);
+        } else { //hex string, e.g. "FF0000"
+          byte brgbw[] = {0,0,0,0};
+          const char* hexCol = iarr[i];
+          if (colorFromHexString(brgbw, hexCol)) {
+            for (uint8_t c = 0; c < 4; c++) rgbw[c] = brgbw[c];
+          }
         }
-        if (!set) start++;
-        set = 0;
       }
     }
     strip.setPixelSegment(255);
@@ -204,7 +201,7 @@ void deserializeSegment(JsonObject elem, byte it, byte presetId)
   } else { //return to regular effect
     seg.setOption(SEG_OPTION_FREEZE, false);
   }
-  return; // seg.hasChanged(prev);
+  return; // seg.differs(prev);
 }
 
 bool deserializeState(JsonObject root, byte callMode, byte presetId)
@@ -351,9 +348,9 @@ bool deserializeState(JsonObject root, byte callMode, byte presetId)
   }
 
   JsonObject playlist = root[F("playlist")];
-  if (!playlist.isNull()) {
-    loadPlaylist(playlist, presetId);
-    noNotification = true; //do not notify both for this request and the first playlist entry
+  if (!playlist.isNull() && loadPlaylist(playlist, presetId)) {
+    //do not notify here, because the first playlist entry will do
+    noNotification = true;
   } else {
     interfaceUpdateCallMode = CALL_MODE_WS_SEND;
   }

wled00/playlist.cpp

@@ -53,16 +53,16 @@ void unloadPlaylist() {
 }
 
 
-void loadPlaylist(JsonObject playlistObj, byte presetId) {
+int16_t loadPlaylist(JsonObject playlistObj, byte presetId) {
   unloadPlaylist();
   
   JsonArray presets = playlistObj["ps"];
   playlistLen = presets.size();
-  if (playlistLen == 0) return;
+  if (playlistLen == 0) return -1;
   if (playlistLen > 100) playlistLen = 100;
 
   playlistEntries = new PlaylistEntry[playlistLen];
-  if (playlistEntries == nullptr) return;
+  if (playlistEntries == nullptr) return -1;
 
   byte it = 0;
   for (int ps : presets) {
@@ -113,6 +113,7 @@ void loadPlaylist(JsonObject playlistObj, byte presetId) {
 
   currentPlaylist = presetId;
   DEBUG_PRINTLN(F("Playlist loaded."));
+  return currentPlaylist;
 }
 
 

wled00/wled.h

@@ -3,12 +3,12 @@
 /*
    Main sketch, global variable declarations
    @title WLED project sketch
-   @version 0.13.0-bl1
+   @version 0.13.0-bl2
    @author Christian Schwinne
  */
 
 // version code in format yymmddb (b = daily build)
-#define VERSION 2107101
+#define VERSION 2107141
 
 //uncomment this if you have a "my_config.h" file you'd like to use
 //#define WLED_USE_MY_CONFIG
@@ -499,7 +499,7 @@ WLED_GLOBAL bool blynkEnabled _INIT(false);
 
 //playlists
 WLED_GLOBAL unsigned long presetCycledTime _INIT(0);
-WLED_GLOBAL int16_t currentPlaylist _INIT(0);
+WLED_GLOBAL int16_t currentPlaylist _INIT(-1);
 //still used for "PL=~" HTTP API command
 WLED_GLOBAL byte presetCycleMin _INIT(1), presetCycleMax _INIT(5);
 WLED_GLOBAL byte presetCycCurr _INIT(presetCycleMin);

wled00/ws.cpp

@@ -38,13 +38,20 @@ void wsEvent(AsyncWebSocket * server, AsyncWebSocketClient * client, AwsEventTyp
             serializeJson(root,Serial);
             DEBUG_PRINTLN();
           #endif
-          if (root.containsKey("lv"))
+          if (root["v"] && root.size() == 1) {
+            //if the received value is just "{"v":true}", send only to this client
+            verboseResponse = true;
+          } else if (root.containsKey("lv"))
           {
             wsLiveClientId = root["lv"] ? client->id() : 0;
           } else {
             fileDoc = &jsonBuffer;
             verboseResponse = deserializeState(root);
             fileDoc = nullptr;
+            if (!interfaceUpdateCallMode) {
+              //special case, only on playlist load, avoid sending twice in rapid succession
+              if (millis() - lastInterfaceUpdate > 1700) verboseResponse = false;
+            }
           }
         }
         //update if it takes longer than 300ms until next "broadcast"