DarthAffe/Artemis
1
0
Fork 0
mirror of https://github.com/Artemis-RGB/Artemis synced 2025-12-13 05:48:35 +00:00
Code Issues Releases Wiki Activity

RGB Service - Added verbose logging

Browse Source
This commit is contained in:
Robert 2022-11-06 10:16:18 +01:00
parent a66cc26f29
commit ff633f7e8e
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Artemis.Core/Services/RgbService.cs
View File

@ -170,11 +170,13 @@ internal class RgbService : IRgbService
public void AddDeviceProvider(IRGBDeviceProvider deviceProvider) public void AddDeviceProvider(IRGBDeviceProvider deviceProvider)
{ {
_logger.Verbose("[AddDeviceProvider] Pausing rendering to add {DeviceProvider}", deviceProvider.GetType().Name);
bool changedRenderPaused = SetRenderPaused(true); bool changedRenderPaused = SetRenderPaused(true);
try try
{ {
List<ArtemisDevice> toRemove = _devices.Where(a => deviceProvider.Devices.Any(d => a.RgbDevice == d)).ToList(); List<ArtemisDevice> toRemove = _devices.Where(a => deviceProvider.Devices.Any(d => a.RgbDevice == d)).ToList();
_logger.Verbose("[AddDeviceProvider] Removing {Count} old device(s)", toRemove.Count);
Surface.Detach(toRemove.Select(d => d.RgbDevice)); Surface.Detach(toRemove.Select(d => d.RgbDevice));
foreach (ArtemisDevice device in toRemove) foreach (ArtemisDevice device in toRemove)
RemoveDevice(device); RemoveDevice(device);
@ -189,8 +191,10 @@ internal class RgbService : IRgbService
_logger.Warning(e.Exception, "Device provider {deviceProvider} threw non-critical exception", deviceProvider.GetType().Name); _logger.Warning(e.Exception, "Device provider {deviceProvider} threw non-critical exception", deviceProvider.GetType().Name);
} }
_logger.Verbose("[AddDeviceProvider] Initializing device provider");
deviceProvider.Exception += DeviceProviderOnException; deviceProvider.Exception += DeviceProviderOnException;
deviceProvider.Initialize(); deviceProvider.Initialize();
_logger.Verbose("[AddDeviceProvider] Attaching devices of device provider");
Surface.Attach(deviceProvider.Devices); Surface.Attach(deviceProvider.Devices);
deviceProvider.Exception -= DeviceProviderOnException; deviceProvider.Exception -= DeviceProviderOnException;
if (providerExceptions.Count == 1) if (providerExceptions.Count == 1)
@ -220,7 +224,10 @@ internal class RgbService : IRgbService
} }
finally finally
{ {
_logger.Verbose("[AddDeviceProvider] Updating the LED group");
UpdateLedGroup(); UpdateLedGroup();
_logger.Verbose("[AddDeviceProvider] Resuming rendering after adding {DeviceProvider}", deviceProvider.GetType().Name);
if (changedRenderPaused) if (changedRenderPaused)
SetRenderPaused(false); SetRenderPaused(false);
} }
@ -228,11 +235,13 @@ internal class RgbService : IRgbService
public void RemoveDeviceProvider(IRGBDeviceProvider deviceProvider) public void RemoveDeviceProvider(IRGBDeviceProvider deviceProvider)
{ {
_logger.Verbose("[RemoveDeviceProvider] Pausing rendering to remove {DeviceProvider}", deviceProvider.GetType().Name);
bool changedRenderPaused = SetRenderPaused(true); bool changedRenderPaused = SetRenderPaused(true);
try try
{ {
List<ArtemisDevice> toRemove = _devices.Where(a => deviceProvider.Devices.Any(d => a.RgbDevice == d)).ToList(); List<ArtemisDevice> toRemove = _devices.Where(a => deviceProvider.Devices.Any(d => a.RgbDevice == d)).ToList();
_logger.Verbose("[RemoveDeviceProvider] Removing {Count} old device(s)", toRemove.Count);
Surface.Detach(toRemove.Select(d => d.RgbDevice)); Surface.Detach(toRemove.Select(d => d.RgbDevice));
foreach (ArtemisDevice device in toRemove) foreach (ArtemisDevice device in toRemove)
RemoveDevice(device); RemoveDevice(device);
@ -246,7 +255,10 @@ internal class RgbService : IRgbService
} }
finally finally
{ {
_logger.Verbose("[RemoveDeviceProvider] Updating the LED group");
UpdateLedGroup(); UpdateLedGroup();
_logger.Verbose("[RemoveDeviceProvider] Resuming rendering after adding {DeviceProvider}", deviceProvider.GetType().Name);
if (changedRenderPaused) if (changedRenderPaused)
SetRenderPaused(false); SetRenderPaused(false);
} }