Remove Trace.WriteLine from RemoveInitializersFromInputs (#6)

nietras · Jun 13, 2023 · 7862453 · 7862453
1 parent 9dc908d
commit 7862453
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -2,15 +2,16 @@
 <Project>
   <PropertyGroup>
     <Authors>nietras</Authors>
-    <Copyright>Copyright © nietras 2021</Copyright>
+    <Copyright>Copyright © nietras 2023</Copyright>
     <NeutralLanguage>en</NeutralLanguage>
     <AssemblyVersion>0.2.0.0</AssemblyVersion>
-    <FileVersion>0.2.0</FileVersion>
+    <FileVersion>0.2.1</FileVersion>
     <InformationalVersion>$(FileVersion)</InformationalVersion>
     <PackageVersion>$(InformationalVersion)</PackageVersion>
 
     <!--https://blog.paranoidcoding.com/2016/04/05/deterministic-builds-in-roslyn.html-->
     <Deterministic>true</Deterministic>
     <LangVersion>9.0</LangVersion>
+    <CheckEolTargetFramework>false</CheckEolTargetFramework>
   </PropertyGroup>
 </Project>
diff --git a/src/OnnxSharp/GraphExtensions.Clean.cs b/src/OnnxSharp/GraphExtensions.Clean.cs
@@ -29,7 +29,7 @@ public static void RemoveInitializersFromInputs(this GraphProto graph)
                 {
                     // https://github.com/protocolbuffers/protobuf/blob/master/csharp/src/Google.Protobuf/Collections/RepeatedField.cs
                     var removed = inputs.Remove(input);
-                    Trace.WriteLine($"{removed} {inputs.Count}");
+                    Debug.Assert(removed, $"{removed} {inputs.Count}");
                 }
             }
         }
@@ -55,7 +55,7 @@ public static void RemoveUnnecessaryInitializerReshapes(this GraphProto graph)
                     var outputs = node.Output;
 
                     // Expected Reshape takes 2 inputs and has 1 output
-                    if (inputs.Count == opSpec.Inputs && outputs.Count == opSpec.Outputs) 
+                    if (inputs.Count == opSpec.Inputs && outputs.Count == opSpec.Outputs)
                     {
                         var dataName = inputs[0];
                         var shapeName = inputs[1];
@@ -70,7 +70,7 @@ public static void RemoveUnnecessaryInitializerReshapes(this GraphProto graph)
                             var outputShapeValue = valueInfos.Single(v => v.Name, reshapeOutputName);
 
                             var outputShapeDims = outputShapeValue.Type.TensorType.Shape.Dim;
-                            var allValue = outputShapeDims.All(d => d.ValueCase == 
+                            var allValue = outputShapeDims.All(d => d.ValueCase ==
                                 TensorShapeProto.Types.Dimension.ValueOneofCase.DimValue);
                             if (allValue)
                             {