Working on render logic

src/Artemis.Core/Services/CoreService.cs

@@ -64,12 +64,17 @@ namespace Artemis.Core.Services
                 foreach (var module in modules)
                     module.Update(args.DeltaTime);
 
+                // If there is no graphics decorator, skip the frame
                 if (_rgbService.GraphicsDecorator == null)
                     return;
 
                 // Render all active modules
                 using (var g = _rgbService.GraphicsDecorator.GetGraphics())
                 {
+                    // If there are no graphics, skip the frame
+                    if (g == null)
+                        return;
+
                     g.Clear(Color.Black);
                     foreach (var module in modules)
                         module.Render(args.DeltaTime, _rgbService.Surface, g);

src/Artemis.Core/Services/Interfaces/IRgbService.cs

@@ -25,5 +25,7 @@ namespace Artemis.Core.Services.Interfaces
         ///     Occurs when a single device has reloaded
         /// </summary>
         event EventHandler<DeviceEventArgs> DeviceReloaded;
+
+        void UpdateGraphicsDecorator();
     }
 }

src/Artemis.Core/Services/RgbService.cs

@@ -19,6 +19,7 @@ namespace Artemis.Core.Services
         private readonly List<IRGBDevice> _loadedDevices;
         private readonly ILogger _logger;
         private readonly TimerUpdateTrigger _updateTrigger;
+        private ListLedGroup _background;
 
         internal RgbService(ILogger logger)
         {
@@ -58,7 +59,7 @@ namespace Artemis.Core.Services
                 _logger.Warning("Device provider {deviceProvider} has no devices", deviceProvider.GetType().Name);
                 return;
             }
-            
+
             lock (_loadedDevices)
             {
                 foreach (var surfaceDevice in deviceProvider.Devices)
@@ -74,11 +75,6 @@ namespace Artemis.Core.Services
                     }
                 }
             }
-
-            // Apply the application wide brush and decorator
-            var background = new ListLedGroup(Surface.Leds) {Brush = new SolidColorBrush(new Color(255, 255, 255, 255))};
-            GraphicsDecorator = new GraphicsDecorator(background);
-            background.Brush.AddDecorator(GraphicsDecorator);
         }
 
         public void Dispose()
@@ -99,6 +95,23 @@ namespace Artemis.Core.Services
         public event EventHandler<DeviceEventArgs> DeviceLoaded;
         public event EventHandler<DeviceEventArgs> DeviceReloaded;
 
+        public void UpdateGraphicsDecorator()
+        {
+            // Clean up the old background if present
+            if (_background != null)
+            {
+                _background.RemoveAllDecorators();
+                _background.Detach();
+            }
+
+            // Apply the application wide brush and decorator
+            _background = new ListLedGroup(Surface.Leds) {Brush = new SolidColorBrush(new Color(255, 255, 255, 255))};
+            GraphicsDecorator = new GraphicsDecorator(_background);
+            _background.Brush.RemoveAllDecorators();
+
+            _background.Brush.AddDecorator(GraphicsDecorator);
+        }
+
         private void OnDeviceLoaded(DeviceEventArgs e)
         {
             DeviceLoaded?.Invoke(this, e);

src/Artemis.Core/Services/Storage/SurfaceService.cs

@@ -58,7 +58,7 @@ namespace Artemis.Core.Services.Storage
                         deviceConfiguration.ApplyToDevice();
                 }
                 // Update the RGB service's graphics decorator to work with the new surface configuration
-                _rgbService.GraphicsDecorator.UpdateBitmap();
+                _rgbService.UpdateGraphicsDecorator();
 
                 OnActiveSurfaceConfigurationChanged(new SurfaceConfigurationEventArgs(_activeSurfaceConfiguration));
             }