nettika/mattercontrol
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make sure we can build spiral vase even if the number of perimeters is 0

Browse source 
issue: MatterHackers/MCCentral#6245
0% infill and 0 perimeters should trigger warning in MC
This commit is contained in:
Lars Brubaker 2021-01-26 17:26:10 -08:00
parent c81636c654
commit 5f8ffe5650
3 changed files with 30 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

28
MatterControlLib/ApplicationView/SettingsValidation.cs
View file

@ -370,6 +370,34 @@ namespace MatterHackers.MatterControl
});
}
if (settings.GetValue<double>(SettingsKey.fill_density) <= 0)
{
errors.Add(
new SettingsValidationError(SettingsKey.fill_density)
{
Error = "{0} should be greater than 0.".Localize().FormatWith(
GetSettingsName(SettingsKey.fill_density)),
ErrorLevel = ValidationErrorLevel.Warning,
ValueDetails = "{0} = {1}".FormatWith(
GetSettingsName(SettingsKey.fill_density),
settings.GetValue<double>(SettingsKey.fill_density)),
});
}
if (settings.GetValue<double>(SettingsKey.perimeters) <= 0)
{
errors.Add(
new SettingsValidationError(SettingsKey.perimeters)
{
Error = "{0} should be greater than 0.".Localize().FormatWith(
GetSettingsName(SettingsKey.perimeters)),
ErrorLevel = ValidationErrorLevel.Warning,
ValueDetails = "{0} = {1}".FormatWith(
GetSettingsName(SettingsKey.perimeters),
settings.GetValue<double>(SettingsKey.perimeters)),
});
}
if (settings.GetValue<double>(SettingsKey.infill_overlap_perimeter) < -settings.GetValue<double>(SettingsKey.nozzle_diameter)
|| settings.GetValue<double>(SettingsKey.infill_overlap_perimeter) > settings.GetValue<double>(SettingsKey.nozzle_diameter))
{

2
Submodules/MatterSlice

@ -1 +1 @@
Subproject commit b35dfe02a31e3d3c328ab68ebd4c99f50d7759c7
Subproject commit 4f71564d218f924f454e5b20c11001f7a1a22d79

2
Submodules/agg-sharp

@ -1 +1 @@
Subproject commit 55d36cd0c334d05389b56d65303462bc309ede2b
Subproject commit 3901db65a244d449c0aa33b992184eec62953ae7