WildernessLabs · jorgedevs · May 29, 2024 · May 28, 2024
diff --git a/.github/workflows/main-ci.yml b/.github/workflows/main-ci.yml
@@ -3,7 +3,6 @@ name: Main Build
 on:
   workflow_dispatch:
   pull_request:
-    branches: [ main ]
   push:
     branches: [ main ]
 
@@ -18,6 +17,7 @@ jobs:
       uses: actions/checkout@v3
       with:
         path: Meadow.Foundation.FeatherWings
+        ref: main
 
     - name: Setup .NET SDK
       uses: actions/setup-dotnet@v1

diff --git a/Source/CharlieWing/Driver/CharlieWing.cs b/Source/CharlieWing/Driver/CharlieWing.cs
@@ -10,10 +10,14 @@ namespace Meadow.Foundation.FeatherWings
     /// </summary>
     public class CharlieWing : IPixelDisplay
     {
+        private const int WidthInPixels = 15;
+        private const int HeightInPixels = 7;
+        private const int MaxFrames = 7;
+
         /// <summary>
-        /// Is31fl3731 object to manage the leds
+        /// Is31fl3731 object to manage the LEDs
         /// </summary>
-        protected readonly Is31fl3731 iS31FL3731;
+        protected readonly Is31fl3731 is31Fl3731;
 
         /// <summary>
         /// Color mode of display
@@ -23,12 +27,12 @@ public class CharlieWing : IPixelDisplay
         /// <summary>
         /// Width of display in pixels
         /// </summary>
-        public int Width => 15;
+        public int Width => WidthInPixels;
 
         /// <summary>
         /// Height of display in pixels
         /// </summary>
-        public int Height => 7;
+        public int Height => HeightInPixels;
 
         /// <summary>
         /// The Is31fl3731 active frame 
@@ -53,13 +57,13 @@ public class CharlieWing : IPixelDisplay
         /// <param name="address">The I2C address</param>
         public CharlieWing(II2cBus i2cBus, byte address = (byte)Is31fl3731.Addresses.Default)
         {
-            iS31FL3731 = new Is31fl3731(i2cBus, address);
-            iS31FL3731.Initialize();
+            is31Fl3731 = new Is31fl3731(i2cBus, address);
+            is31Fl3731.Initialize();
 
-            for (byte i = 0; i <= 7; i++)
+            for (byte i = 0; i <= MaxFrames; i++)
             {
-                iS31FL3731.SetLedState(i, true);
-                iS31FL3731.Clear(i);
+                is31Fl3731.SetLedState(i, true);
+                is31Fl3731.Clear(i);
             }
         }
 
@@ -69,7 +73,7 @@ public CharlieWing(II2cBus i2cBus, byte address = (byte)Is31fl3731.Addresses.Def
         /// <param name="updateDisplay">Force a display update if true, false to clear the buffer</param>
         public void Clear(bool updateDisplay = false)
         {
-            iS31FL3731.Clear(Frame);
+            is31Fl3731.Clear(Frame);
         }
 
         /// <summary>
@@ -102,6 +106,11 @@ public void DrawPixel(int x, int y, bool colored)
         /// <param name="brightness">The led brightness from 0-255</param>
         public void DrawPixel(int x, int y, byte brightness)
         {
+            if (x < 0 || x >= WidthInPixels || y < 0 || y >= HeightInPixels)
+            {
+                throw new ArgumentOutOfRangeException($"Pixel coordinates ({x}, {y}) are out of bounds.");
+            }
+
             if (x > 7)
             {
                 x = 15 - x;
@@ -112,10 +121,10 @@ public void DrawPixel(int x, int y, byte brightness)
                 y = 7 - y;
             }
 
-            //Swap
+            // Swap
             (y, x) = (x, y);
 
-            iS31FL3731.SetLedPwm(Frame, (byte)(x + y * 16), brightness);
+            is31Fl3731.SetLedPwm(Frame, (byte)(x + y * 16), brightness);
         }
 
         /// <summary>
@@ -125,7 +134,9 @@ public void DrawPixel(int x, int y, byte brightness)
         /// <param name="y">The y position in pixels 0 indexed from the top</param>
         public void InvertPixel(int x, int y)
         {
-            throw new NotImplementedException();
+            byte currentBrightness = is31Fl3731.GetLedPwm(Frame, (byte)(x + y * 16));
+            byte invertedBrightness = (byte)(255 - currentBrightness);
+            DrawPixel(x, y, invertedBrightness);
         }
 
         /// <summary>
@@ -185,7 +196,7 @@ public void Fill(int x, int y, int width, int height, Color fillColor)
         /// </summary>
         public void Show()
         {
-            iS31FL3731.DisplayFrame(Frame);
+            is31Fl3731.DisplayFrame(Frame);
         }
 
         /// <summary>
@@ -201,12 +212,12 @@ public void Show(int left, int top, int right, int bottom)
         }
 
         /// <summary>
-        /// Update the display from a specific iS31FL3731 frame
+        /// Update the display from a specific Is31fl3731 frame
         /// </summary>
         /// <param name="frame">The frame to show (0-7)</param>
         public void Show(byte frame)
         {
-            iS31FL3731.DisplayFrame(frame);
+            is31Fl3731.DisplayFrame(frame);
         }
     }
-}
+}
diff --git a/Source/CharlieWing/Driver/CharlieWing.csproj b/Source/CharlieWing/Driver/CharlieWing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Is31fl3731" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Is31fl3731" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/CharlieWing/Samples/CharlieWing_Sample/CharlieWing_Sample.csproj b/Source/CharlieWing/Samples/CharlieWing_Sample/CharlieWing_Sample.csproj
@@ -6,8 +6,8 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\CharlieWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/DotstarWing/Driver/DotstarWing.csproj b/Source/DotstarWing/Driver/DotstarWing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Leds.Apa102" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Leds.Apa102" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/DotstarWing/Samples/DotstarWing_Sample/DotstarWing_Sample.csproj b/Source/DotstarWing/Samples/DotstarWing_Sample/DotstarWing_Sample.csproj
@@ -6,8 +6,8 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\DotstarWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/GPSWing/Driver/GPSWing.csproj b/Source/GPSWing/Driver/GPSWing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Sensors.Gnss.Mt3339" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Sensors.Gnss.Mt3339" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/GPSWing/Samples/GPSWing_Sample/GPSWing_Sample.csproj b/Source/GPSWing/Samples/GPSWing_Sample/GPSWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
 	<ProjectReference Include="..\..\Driver\GPSWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/KeyboardWing/Driver/KeyboardWing.csproj b/Source/KeyboardWing/Driver/KeyboardWing.csproj
@@ -24,8 +24,8 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Displays.TftSpi" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Sensors.Hid.Bbq10Keyboard" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Sensors.Hid.Tsc2004" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Displays.TftSpi" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Sensors.Hid.Bbq10Keyboard" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Sensors.Hid.Tsc2004" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/KeyboardWing/Samples/KeyboardWing_Sample/KeyboardWing_Sample.csproj b/Source/KeyboardWing/Samples/KeyboardWing_Sample/KeyboardWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\KeyboardWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/LedMatrix8x16Wing/Driver/LedMatrix8x16Wing.csproj b/Source/LedMatrix8x16Wing/Driver/LedMatrix8x16Wing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Ht16k33" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Ht16k33" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/LedMatrix8x16Wing/Samples/LedMatrix8x16Wing_Sample/LedMatrix8x16Wing_Sample.csproj b/Source/LedMatrix8x16Wing/Samples/LedMatrix8x16Wing_Sample/LedMatrix8x16Wing_Sample.csproj
@@ -6,8 +6,8 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\LedMatrix8x16Wing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/MotorWing/Driver/MotorWing.csproj b/Source/MotorWing/Driver/MotorWing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Pca9685" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Pca9685" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/MotorWing/Sample/MotorWing_Sample/MotorWing_Sample.csproj b/Source/MotorWing/Sample/MotorWing_Sample/MotorWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\MotorWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/NeoPixelWing/Driver/NeoPixelWing.csproj b/Source/NeoPixelWing/Driver/NeoPixelWing.csproj
@@ -24,7 +24,7 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Leds.Ws2812" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Leds.Ws2812" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/NeoPixelWing/Sample/NeoPixelWing_Sample/NeoPixelWing_Sample.csproj b/Source/NeoPixelWing/Sample/NeoPixelWing_Sample/NeoPixelWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\NeoPixelWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/NineDofImuWing/Driver/NineDofImuWing.csproj b/Source/NineDofImuWing/Driver/NineDofImuWing.csproj
@@ -24,7 +24,7 @@
 	<None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Sensors.Motion.Lis3mdl" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Sensors.Motion.Lsm6dsox" Version="1.11.0" />
+    <ProjectReference Include="..\..\..\..\Meadow.Foundation\Source\Meadow.Foundation.Peripherals\Sensors.Motion.Lis3Mdl\Driver\Sensors.Motion.Lis3Mdl.csproj" />
+    <ProjectReference Include="..\..\..\..\Meadow.Foundation\Source\Meadow.Foundation.Peripherals\Sensors.Motion.Lsm6Dsox\Driver\Sensors.Motion.Lsm6Dsox.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/NineDofImuWing/Sample/NineDofImuWing_Sample/NineDofImuWing_Sample.csproj b/Source/NineDofImuWing/Sample/NineDofImuWing_Sample/NineDofImuWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\NineDofImuWing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/OLED128x32Wing/Driver/OLED128x32Wing.csproj b/Source/OLED128x32Wing/Driver/OLED128x32Wing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Displays.Ssd130x" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Displays.Ssd130x" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/OLED128x32Wing/Sample/OLED128x32Wing_Sample/OLED128x32Wing_Sample.csproj b/Source/OLED128x32Wing/Sample/OLED128x32Wing_Sample/OLED128x32Wing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\OLED128x32Wing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/OLED128x64Wing/Driver/OLED128x64Wing.csproj b/Source/OLED128x64Wing/Driver/OLED128x64Wing.csproj
@@ -24,6 +24,6 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.Displays.Sh110x" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.Displays.Sh110x" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/OLED128x64Wing/Sample/OLED128x64Wing_Sample/OLED128x64Wing_Sample.csproj b/Source/OLED128x64Wing/Sample/OLED128x64Wing_Sample/OLED128x64Wing_Sample.csproj
@@ -6,8 +6,8 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Graphics.MicroGraphics" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\OLED128x64Wing.csproj" />
   </ItemGroup>
 </Project>
diff --git a/Source/ServoWing/Driver/ServoWing.csproj b/Source/ServoWing/Driver/ServoWing.csproj
@@ -24,7 +24,7 @@
     <None Include="..\..\icon.png" Pack="true" PackagePath="" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Pca9685" Version="1.11.0" />
-    <PackageReference Include="Meadow.Foundation.Servos.ServoCore" Version="1.11.0" />
+    <PackageReference Include="Meadow.Foundation.ICs.IOExpanders.Pca9685" Version="1.12.0" />
+    <PackageReference Include="Meadow.Foundation.Servos.ServoCore" Version="1.12.0" />
   </ItemGroup>
 </Project>
diff --git a/Source/ServoWing/Sample/ServoWing_Sample/ServoWing_Sample.csproj b/Source/ServoWing/Sample/ServoWing_Sample/ServoWing_Sample.csproj
@@ -6,7 +6,7 @@
     <AssemblyName>App</AssemblyName>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="Meadow.F7" Version="1.11.0" />
+    <PackageReference Include="Meadow.F7" Version="1.12.0" />
     <ProjectReference Include="..\..\Driver\ServoWing.csproj" />
   </ItemGroup>
 </Project>