LA-216 display options

[dbgrande]

LA-216 is a nice addition to your collection of modules.

Besides the default to overlay both inputs of each channel (left), here are a couple of other possibilities for you to consider:

• Interleave the first 8 channels of both inputs (middle).

• Offset the first 8 channels of input 2 (right).

[dbgrande]

FYI, here are the code changes I did to implement the above:

--- LA2.cpp.orig	2021-12-03 18:13:09.018692897 -0700
+++ LA2.cpp	2021-12-11 13:32:06.617302802 -0700
@@ -17,6 +17,7 @@
 		PARAM_RESET,
 		PARAM_PRE,
 		PARAM_COLORS,
+		PARAM_DMODE,
 		NUM_PARAMS
 	};
 	enum InputIds {
@@ -61,6 +62,7 @@
 		configParam(PARAM_INDEX_2, 0.0f, 1.0f, 1.0f, "Right Index Position", "%", 0.0f, 100.0f, 0.0f);
 		configParam(PARAM_PRE, 0.0f, 32.0f, 0.0f, "Pre-trigger Buffer Size", " Points");
 		configSwitch(PARAM_COLORS, 0, 1, 0, "Match cable colors", { "Off", "On" });
+		configSwitch(PARAM_DMODE, 0, 2, 0, "Display mode", { "Overlay 16", "Interleave 8", "Offset 8" });
 		configInput(INPUT_1, "Signal 1");
 		configInput(INPUT_2, "Signal 2");
 		configInput(INPUT_EXT, "External Trigger");
@@ -327,11 +329,15 @@
 				return;
 			}
 			if (layer == 1) {
+				int dmode = clamp(module->params[LA_216::PARAM_DMODE].getValue(), 0.0f, 2.0f);
+				const int dchans[3] = { 16, 8, 8 };
+				const int dstep[3] = { 22, 44, 22 };
+				const int doffset[3] = { 0, 22, 176 };
 				for (int i = 0; i < 2; i++) {
 					if (module->inputs[LA_216::INPUT_1 + i].isConnected()) {
 						NVGcolor col = getColor(i);
-						for (int c = 0; c < module->inputs[LA_216::INPUT_1 + i].getChannels(); c++) {
-							drawTrace(args.vg, module->buffer[i * 16 + c], 21.5f + 22 * c, col); 
+						for (int c = 0; c < std::min(dchans[dmode], module->inputs[LA_216::INPUT_1 + i].getChannels()); c++) {
+							drawTrace(args.vg, module->buffer[i * 16 + c], 21.5f + c * dstep[dmode] + i * doffset[dmode], col);
 						}
 					}
 				}
@@ -434,6 +440,7 @@
 		addParam(createParamCentered<SmallKnob<LightKnob>>(Vec(18.5, 337), module, LA_216::PARAM_INDEX_1));
 		addParam(createParamCentered<SmallKnob<LightKnob>>(Vec(46.5, 337), module, LA_216::PARAM_INDEX_2));
 		addParam(createParamCentered<SnapKnob<SmallKnob<LightKnob>>>(Vec(48.5, 163), module, LA_216::PARAM_PRE));
+		addParam(createParamCentered<SubSwitch3>(Vec(54, 115.5), module, LA_216::PARAM_DMODE));
 	}
 	void appendContextMenu(Menu *menu) override {
 		SchemeModuleWidget::appendContextMenu(menu);

[firolightfog]

Oh, it's a nice one! But this change hasn't made it to the official Library release, has it?