diff --git a/light9/collector/collector.py b/light9/collector/collector.py
--- a/light9/collector/collector.py
+++ b/light9/collector/collector.py
@@ -214,7 +214,7 @@ class Collector:
                 index = DmxMessageIndex(_index)
                 outArray = pendingOut[outputUri]
                 if outArray[index] != 0:
-                    log.warn(f'conflict: {outputUri} output array was already nonzero at 0-based index {index}')
+                    log.warning(f'conflict: {outputUri} output array was already nonzero at 0-based index {index}')
                     raise ValueError(f"someone already wrote to index {index}")
                 outArray[index] = value
         return pendingOut
diff --git a/light9/effect/settings.py b/light9/effect/settings.py
--- a/light9/effect/settings.py
+++ b/light9/effect/settings.py
@@ -288,7 +288,7 @@ class DeviceSettings(_Settings):
 
 @dataclass
 class BareEffectSettings:
-    # settings not specific to any effect
+    # settings for an already-selected EffectClass
     s: Dict[EffectAttr, VTUnion]
 
     def withStrength(self, strength: float) -> BareEffectSettings:
diff --git a/light9/effect/simple_outputs_test.py b/light9/effect/simple_outputs_test.py
new file mode 100644