Prevent boon sim from parseing negatives and added to settings for 10…

…s/30s graphs
baaron4 · Jun 19, 2018 · 67beb06 · EliphasNUIT · Jun 19, 2018 · 67beb06
1 parent f0e9d11
commit 67beb06
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 18 deletions.
diff --git a/LuckParser/App.config b/LuckParser/App.config
@@ -68,10 +68,10 @@
                 <value>True</value>
             </setting>
             <setting name="Show10s" serializeAs="String">
-              <value>True</value>
+                <value>True</value>
             </setting>
             <setting name="Show30s" serializeAs="String">
-              <value>True</value>
+                <value>True</value>
             </setting>
         </LuckParser.Properties.Settings>
     </userSettings>

diff --git a/LuckParser/Models/ParseModels/Players/AbstractPlayer.cs b/LuckParser/Models/ParseModels/Players/AbstractPlayer.cs
@@ -292,8 +292,12 @@ private void setBoonDistribution(BossData bossData, SkillData skillData, List<Co
 
                         for (int i = start; i <= end; i++)
                         {
-                            toFill[i] = new Point(i, simul.getStack(i));
-                            toFillPresence[i] = new Point(i, simul.getItemDuration() > 0 ? 1 : 0);
+                            if (i >= 0)
+                            {
+                                toFill[i] = new Point(i, simul.getStack(i));
+                                toFillPresence[i] = new Point(i, simul.getItemDuration() > 0 ? 1 : 0);
+                            }
+
                         }
                     }
                     // reduce precision to seconds

diff --git a/LuckParser/Properties/Settings.Designer.cs b/LuckParser/Properties/Settings.Designer.cs
diff --git a/LuckParser/Properties/Settings.settings b/LuckParser/Properties/Settings.settings
@@ -59,5 +59,11 @@
     <Setting Name="ParsePhases" Type="System.Boolean" Scope="User">
       <Value Profile="(Default)">True</Value>
     </Setting>
+    <Setting Name="Show10s" Type="System.Boolean" Scope="User">
+      <Value Profile="(Default)">True</Value>
+    </Setting>
+    <Setting Name="Show30s" Type="System.Boolean" Scope="User">
+      <Value Profile="(Default)">True</Value>
+    </Setting>
   </Settings>
 </SettingsFile>