From 73565ae691b409af9190a02485e51ed590510384 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Mon, 9 Jul 2018 20:59:46 -0400 Subject: fix stop count --- lib/stitch_plan/stitch_plan.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 93bcd195..742916f0 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -183,10 +183,7 @@ class ColorBlock(object): def num_stops(self): """Number of pauses in this color block.""" - # Stops are encoded using two STOP stitches each. See the comment in - # stop.py for an explanation. - - return sum(1 for stitch in self if stitch.stop) / 2 + return sum(1 for stitch in self if stitch.stop) @property def num_trims(self): -- cgit v1.3.1 From 2cd4963d09ef78dd25a7401cc47a69474d7fa952 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Sun, 15 Jul 2018 22:53:18 -0400 Subject: adjust stitch plan code for pyembroidery --- lib/output.py | 6 ++- lib/stitch_plan/stitch.py | 14 ++++++- lib/stitch_plan/stitch_plan.py | 94 +++++++++++++++++++++++++----------------- lib/stitch_plan/stop.py | 82 +++++++++++++++++++++++++----------- lib/stitch_plan/ties.py | 18 ++++---- lib/stitch_plan/trim.py | 23 ----------- 6 files changed, 140 insertions(+), 97 deletions(-) delete mode 100644 lib/stitch_plan/trim.py (limited to 'lib/stitch_plan') diff --git a/lib/output.py b/lib/output.py index 491c190a..1c580f04 100644 --- a/lib/output.py +++ b/lib/output.py @@ -111,7 +111,11 @@ def write_embroidery_file(file_path, stitch_plan, svg): # convert from pixels to millimeters # also multiply by 10 to get tenths of a millimeter as required by pyembroidery - "scale": (scale, scale) + "scale": (scale, scale), + + # This forces a jump at the start of the design and after each trim, + # even if we're close enough not to need one. + "full_jump": True, } pyembroidery.write(pattern, file_path, settings) diff --git a/lib/stitch_plan/stitch.py b/lib/stitch_plan/stitch.py index 12642a60..5230efec 100644 --- a/lib/stitch_plan/stitch.py +++ b/lib/stitch_plan/stitch.py @@ -2,7 +2,7 @@ from ..utils.geometry import Point class Stitch(Point): - def __init__(self, x, y, color=None, jump=False, stop=False, trim=False, color_change=False, no_ties=False): + def __init__(self, x, y, color=None, jump=False, stop=False, trim=False, color_change=False, fake_color_change=False, no_ties=False): self.x = x self.y = y self.color = color @@ -10,10 +10,20 @@ class Stitch(Point): self.trim = trim self.stop = stop self.color_change = color_change + self.fake_color_change = fake_color_change self.no_ties = no_ties def __repr__(self): - return "Stitch(%s, %s, %s, %s, %s, %s, %s)" % (self.x, self.y, self.color, "JUMP" if self.jump else " ", "TRIM" if self.trim else " ", "STOP" if self.stop else " ", "NO TIES" if self.no_ties else " ") + return "Stitch(%s, %s, %s, %s, %s, %s, %s, %s%s)" % (self.x, + self.y, + self.color, + "JUMP" if self.jump else " ", + "TRIM" if self.trim else " ", + "STOP" if self.stop else " ", + "NO TIES" if self.no_ties else " ", + "FAKE " if self.fake_color_change else "", + "COLOR CHANGE" if self.color_change else " " + ) def copy(self): return Stitch(self.x, self.y, self.color, self.jump, self.stop, self.trim, self.color_change, self.no_ties) diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 742916f0..1a466295 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -16,64 +16,40 @@ def patches_to_stitch_plan(patches, collapse_len=3.0 * PIXELS_PER_MM): """ stitch_plan = StitchPlan() - color_block = stitch_plan.new_color_block() - need_trim = False + if not patches: + return stitch_plan + + color_block = stitch_plan.new_color_block(color=patches[0].color) + for patch in patches: if not patch.stitches: continue - if not color_block.has_color(): - # set the color for the first color block - color_block.color = patch.color - - if color_block.color == patch.color: - if need_trim: - process_trim(color_block, patch.stitches[0]) - need_trim = False - - # add a jump stitch between patches if the distance is more - # than the collapse length - if color_block.last_stitch: - if (patch.stitches[0] - color_block.last_stitch).length() > collapse_len: - color_block.add_stitch(patch.stitches[0].x, patch.stitches[0].y, jump=True) - - else: + if color_block.color != patch.color or color_block.stop_after: # add a color change (only if we didn't just do a "STOP after") - if not color_block.last_stitch.color_change: - stitch = color_block.last_stitch.copy() - stitch.color_change = True - color_block.add_stitch(stitch) + if not color_block.stop_after: + color_block.add_stitch(color_change=True) - color_block = stitch_plan.new_color_block() - color_block.color = patch.color + color_block = stitch_plan.new_color_block(color=patch.color) color_block.filter_duplicate_stitches() color_block.add_stitches(patch.stitches, no_ties=patch.stitch_as_is) if patch.trim_after: - # a trim needs to be followed by a jump to the next stitch, so - # we'll process it when we start the next patch - need_trim = True + color_block.add_stitch(trim=True) if patch.stop_after: - process_stop(color_block) + process_stop(stitch_plan) + + # process_stop() may have split the block into two + color_block = stitch_plan.last_color_block - add_jumps(stitch_plan) add_ties(stitch_plan) return stitch_plan -def add_jumps(stitch_plan): - """Add a JUMP stitch at the start of each color block.""" - - for color_block in stitch_plan: - stitch = color_block.stitches[0].copy() - stitch.jump = True - color_block.stitches.insert(0, stitch) - - class StitchPlan(object): """Holds a set of color blocks, each containing stitches.""" @@ -85,6 +61,9 @@ class StitchPlan(object): self.color_blocks.append(color_block) return color_block + def add_color_block(self, color_block): + self.color_blocks.append(color_block) + def __iter__(self): return iter(self.color_blocks) @@ -99,6 +78,10 @@ class StitchPlan(object): """Number of unique colors in the stitch plan.""" return len({block.color for block in self}) + @property + def num_color_blocks(self): + return len(self.color_blocks) + @property def num_stops(self): return sum(block.num_stops for block in self) @@ -137,6 +120,13 @@ class StitchPlan(object): dimensions = self.dimensions return (dimensions[0] / PIXELS_PER_MM, dimensions[1] / PIXELS_PER_MM) + @property + def last_color_block(self): + if self.color_blocks: + return self.color_blocks[-1] + else: + return None + class ColorBlock(object): """Holds a set of stitches, all with the same thread color.""" @@ -148,6 +138,9 @@ class ColorBlock(object): def __iter__(self): return iter(self.stitches) + def __len__(self): + return len(self.stitches) + def __repr__(self): return "ColorBlock(%s, %s)" % (self.color, self.stitches) @@ -191,6 +184,13 @@ class ColorBlock(object): return sum(1 for stitch in self if stitch.trim) + @property + def stop_after(self): + if self.last_stitch is not None: + return self.last_stitch.stop + else: + return False + def filter_duplicate_stitches(self): if not self.stitches: return @@ -212,11 +212,21 @@ class ColorBlock(object): self.stitches = stitches def add_stitch(self, *args, **kwargs): + if not args: + # They're adding a command, e.g. `color_block.add_stitch(stop=True)``. + # Use the position from the last stitch. + if self.last_stitch: + args = (self.last_stitch.x, self.last_stitch.y) + else: + raise ValueError("internal error: can't add a command to an empty stitch block") + if isinstance(args[0], Stitch): self.stitches.append(args[0]) elif isinstance(args[0], Point): self.stitches.append(Stitch(args[0].x, args[0].y, *args[1:], **kwargs)) else: + if not args and self.last_stitch: + args = (self.last_stitch.x, self.last_stitch.y) self.stitches.append(Stitch(*args, **kwargs)) def add_stitches(self, stitches, *args, **kwargs): @@ -237,3 +247,11 @@ class ColorBlock(object): maxy = max(stitch.y for stitch in self) return minx, miny, maxx, maxy + + def split_at(self, index): + """Split this color block into two at the specified stitch index""" + + new_color_block = ColorBlock(self.color, self.stitches[index:]) + del self.stitches[index:] + + return new_color_block diff --git a/lib/stitch_plan/stop.py b/lib/stitch_plan/stop.py index 81dec1da..12a88d3a 100644 --- a/lib/stitch_plan/stop.py +++ b/lib/stitch_plan/stop.py @@ -1,43 +1,75 @@ -def process_stop(color_block): +from ..svg import PIXELS_PER_MM + +def process_stop(stitch_plan): """Handle the "stop after" checkbox. The user wants the machine to pause after this patch. This can be useful for applique and similar on multi-needle machines that normally would not stop between colors. - In machine embroidery files, there's no such thing as an actual - "STOP" instruction. All that exists is a "color change" command - (which libembroidery calls STOP just to be confusing). + In most machine embroidery file formats, there's no such thing as + an actual "STOP" instruction. All that exists is a "color change" + command. On multi-needle machines, the user assigns needles to the colors in - the design before starting stitching. C01, C02, etc are normal + the design before starting stitching. C01, C02, etc are the normal needles, but C00 is special. For a block of stitches assigned to C00, the machine will continue sewing with the last color it - had and pause after it completes the C00 block. + had and pause after it completes the C00 block. Machines that don't + call it C00 still have a similar concept. + + We'll add a STOP instruction at the end of this color block. + Unfortunately, we have a bit of a catch-22: the user needs to set + C00 (or equivalent) for the _start_ of this block to get the + machine to stop at the end of this block. That means it will use + the previous color, which isn't the right color at all! - That means we need to add an artificial color change instruction - shortly before the current stitch so that the user can set that color - block to C00. We'll go back 3 stitches and mark the start of the C00 - block: + For the first STOP in a given thread color, we'll need to + introduce an extra color change. The user can then set the correct + color for the first section and C00 for the second, resulting in + a stop where we want it. + + We'll try to find a logical place to split the color block, like + a TRIM or a really long stitch. Failing that, we'll just split + it in half. """ - if len(color_block.stitches) >= 3: - # make a copy of the stitch and set it as a color change - stitch = color_block.stitches[-3].copy() - stitch.color_change = True + if not stitch_plan.last_color_block or len(stitch_plan.last_color_block) < 2: + return + + last_stitch = stitch_plan.last_color_block.last_stitch + stitch_plan.last_color_block.add_stitch(last_stitch.x, last_stitch.y, stop=True) + + if len(stitch_plan) > 1: + # if this isn't the first stop in this color, then we're done + if stitch_plan.color_blocks[-2].stop_after and \ + stitch_plan.color_blocks[-2].color == stitch_plan.last_color_block.color: + return + + # We need to split this color block. Pick the last TRIM or + # the last long stitch (probably between distant patches). + + for i in xrange(len(stitch_plan.last_color_block) - 2, -1, -1): + stitch = stitch_plan.last_color_block.stitches[i] - # mark this stitch as a "stop" so that we can avoid - # adding tie stitches in ties.py - stitch.stop = True + if stitch.trim: + # ignore the trim right before the stop we just added + if i < len(stitch_plan.last_color_block) - 2: + # split after the trim + i = i + 1 + break - # insert it after the stitch - color_block.stitches.insert(-2, stitch) + if i > 0: + next_stitch = stitch_plan.last_color_block.stitches[i + 1] - # and also add a color change on this stitch, completing the C00 - # block: + if (stitch - next_stitch).length() > 20 * PIXELS_PER_MM: + break - stitch = color_block.stitches[-1].copy() - stitch.color_change = True - color_block.add_stitch(stitch) + if i == 0: + # Darn, we didn't find a TRIM or long stitch. Just chop the + # block in half. + i = len(stitch_plan.last_color_block) / 2 - # reference for the above: https://github.com/lexelby/inkstitch/pull/29#issuecomment-359175447 + new_color_block = stitch_plan.last_color_block.split_at(i) + stitch_plan.last_color_block.add_stitch(color_change=True, fake_color_change=True) + stitch_plan.add_color_block(new_color_block) diff --git a/lib/stitch_plan/ties.py b/lib/stitch_plan/ties.py index 6d07ac71..573469f5 100644 --- a/lib/stitch_plan/ties.py +++ b/lib/stitch_plan/ties.py @@ -30,15 +30,16 @@ def add_tie_in(stitches, upcoming_stitches): def add_ties(stitch_plan): """Add tie-off before and after trims, jumps, and color changes.""" + need_tie_in = True for color_block in stitch_plan: - need_tie_in = True new_stitches = [] for i, stitch in enumerate(color_block.stitches): - # Tie before and after TRIMs, JUMPs, and color changes, but ignore - # the fake color change introduced by a "STOP after" (see stop.py). - is_special = stitch.trim or stitch.jump or (stitch.color_change and not stitch.stop) + is_special = stitch.trim or stitch.jump or stitch.color_change or stitch.stop - if is_special and not need_tie_in: + # see stop.py for an explanation of the fake color change + is_fake = stitch.fake_color_change + + if is_special and not is_fake and not need_tie_in: add_tie_off(new_stitches) new_stitches.append(stitch) need_tie_in = True @@ -49,7 +50,8 @@ def add_ties(stitch_plan): else: new_stitches.append(stitch) - if not need_tie_in: - add_tie_off(new_stitches) - color_block.replace_stitches(new_stitches) + + if not need_tie_in: + # tie off at the end if we haven't already + add_tie_off(color_block.stitches) diff --git a/lib/stitch_plan/trim.py b/lib/stitch_plan/trim.py deleted file mode 100644 index f692a179..00000000 --- a/lib/stitch_plan/trim.py +++ /dev/null @@ -1,23 +0,0 @@ -def process_trim(color_block, next_stitch): - """Handle the "trim after" checkbox. - - DST (and maybe other formats?) has no actual TRIM instruction. - Instead, 3 sequential JUMPs cause the machine to trim the thread. - - To support both DST and other formats, we'll add a TRIM and two - JUMPs. The TRIM will be converted to a JUMP by libembroidery - if saving to DST, resulting in the 3-jump sequence. - """ - - delta = next_stitch - color_block.last_stitch - delta = delta * (1/4.0) - - pos = color_block.last_stitch - - for i in xrange(3): - pos += delta - color_block.add_stitch(pos.x, pos.y, jump=True) - - # first one should be TRIM instead of JUMP - color_block.stitches[-3].jump = False - color_block.stitches[-3].trim = True -- cgit v1.3.1 From 754bf54897e309fa21fa61bc7a626cde71a00f97 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Sun, 15 Jul 2018 23:01:52 -0400 Subject: fix gap caused by splitting block --- lib/stitch_plan/stitch_plan.py | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 1a466295..5d847ad2 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -1,3 +1,5 @@ +from copy import copy + from .stitch import Stitch from .stop import process_stop from .trim import process_trim @@ -254,4 +256,11 @@ class ColorBlock(object): new_color_block = ColorBlock(self.color, self.stitches[index:]) del self.stitches[index:] + # If we're splitting in the middle of a run of stitches, we don't + # want a gap to appear in the preview and the PDF printout, so + # add an extra stitch to bridge the gap. Technically this will + # result in a double needle penetration but it's no big deal. + if not self.last_stitch.trim: + self.add_stitch(copy(new_color_block.stitches[0])) + return new_color_block -- cgit v1.3.1 From b1912157579299212131b86f0b7267d7d91df047 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Mon, 16 Jul 2018 19:38:42 -0400 Subject: tidy up code --- lib/stitch_plan/stitch_plan.py | 9 --------- lib/stitch_plan/stop.py | 35 ++++++++++++++++++++++++----------- 2 files changed, 24 insertions(+), 20 deletions(-) (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 5d847ad2..1a466295 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -1,5 +1,3 @@ -from copy import copy - from .stitch import Stitch from .stop import process_stop from .trim import process_trim @@ -256,11 +254,4 @@ class ColorBlock(object): new_color_block = ColorBlock(self.color, self.stitches[index:]) del self.stitches[index:] - # If we're splitting in the middle of a run of stitches, we don't - # want a gap to appear in the preview and the PDF printout, so - # add an extra stitch to bridge the gap. Technically this will - # result in a double needle penetration but it's no big deal. - if not self.last_stitch.trim: - self.add_stitch(copy(new_color_block.stitches[0])) - return new_color_block diff --git a/lib/stitch_plan/stop.py b/lib/stitch_plan/stop.py index 12a88d3a..0ccaeaf8 100644 --- a/lib/stitch_plan/stop.py +++ b/lib/stitch_plan/stop.py @@ -1,5 +1,8 @@ +from copy import copy + from ..svg import PIXELS_PER_MM + def process_stop(stitch_plan): """Handle the "stop after" checkbox. @@ -34,33 +37,35 @@ def process_stop(stitch_plan): it in half. """ - if not stitch_plan.last_color_block or len(stitch_plan.last_color_block) < 2: + color_block = stitch_plan.last_color_block + + if not color_block or len(color_block) < 2: return - last_stitch = stitch_plan.last_color_block.last_stitch - stitch_plan.last_color_block.add_stitch(last_stitch.x, last_stitch.y, stop=True) + last_stitch = color_block.last_stitch + color_block.add_stitch(stop=True) if len(stitch_plan) > 1: # if this isn't the first stop in this color, then we're done if stitch_plan.color_blocks[-2].stop_after and \ - stitch_plan.color_blocks[-2].color == stitch_plan.last_color_block.color: + stitch_plan.color_blocks[-2].color == color_block.color: return # We need to split this color block. Pick the last TRIM or # the last long stitch (probably between distant patches). - for i in xrange(len(stitch_plan.last_color_block) - 2, -1, -1): - stitch = stitch_plan.last_color_block.stitches[i] + for i in xrange(len(color_block) - 2, -1, -1): + stitch = color_block.stitches[i] if stitch.trim: # ignore the trim right before the stop we just added - if i < len(stitch_plan.last_color_block) - 2: + if i < len(color_block) - 2: # split after the trim i = i + 1 break if i > 0: - next_stitch = stitch_plan.last_color_block.stitches[i + 1] + next_stitch = color_block.stitches[i + 1] if (stitch - next_stitch).length() > 20 * PIXELS_PER_MM: break @@ -68,8 +73,16 @@ def process_stop(stitch_plan): if i == 0: # Darn, we didn't find a TRIM or long stitch. Just chop the # block in half. - i = len(stitch_plan.last_color_block) / 2 + i = len(color_block) / 2 + + new_color_block = color_block.split_at(i) + + # If we're splitting in the middle of a run of stitches, we don't + # want a gap to appear in the preview and the PDF printout, so + # add an extra stitch to bridge the gap. Technically this will + # result in a double needle penetration but it's no big deal. + if not color_block.last_stitch.trim: + color_block.add_stitch(copy(new_color_block.stitches[0])) - new_color_block = stitch_plan.last_color_block.split_at(i) - stitch_plan.last_color_block.add_stitch(color_change=True, fake_color_change=True) + color_block.add_stitch(color_change=True, fake_color_change=True) stitch_plan.add_color_block(new_color_block) -- cgit v1.3.1 From 5ce8df77a07b11b902792d299d4cb89b6951ccd8 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Fri, 20 Jul 2018 21:41:28 -0400 Subject: remove incorrect stop logic --- lib/stitch_plan/stitch_plan.py | 47 +++++++++++----------- lib/stitch_plan/stop.py | 88 ------------------------------------------ 2 files changed, 24 insertions(+), 111 deletions(-) delete mode 100644 lib/stitch_plan/stop.py (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 1a466295..0fa87d71 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -1,6 +1,4 @@ from .stitch import Stitch -from .stop import process_stop -from .trim import process_trim from .ties import add_ties from ..svg import PIXELS_PER_MM from ..utils.geometry import Point @@ -26,26 +24,29 @@ def patches_to_stitch_plan(patches, collapse_len=3.0 * PIXELS_PER_MM): if not patch.stitches: continue - if color_block.color != patch.color or color_block.stop_after: - # add a color change (only if we didn't just do a "STOP after") - if not color_block.stop_after: + if color_block.color != patch.color: + if len(color_block) == 0: + # We just processed a stop, which created a new color block. + # We'll just claim this new block as ours: + color_block.color = patch.color + else: + # end the previous block with a color change color_block.add_stitch(color_change=True) - color_block = stitch_plan.new_color_block(color=patch.color) + # make a new block of our color + color_block = stitch_plan.new_color_block(color=patch.color) - color_block.filter_duplicate_stitches() color_block.add_stitches(patch.stitches, no_ties=patch.stitch_as_is) if patch.trim_after: color_block.add_stitch(trim=True) if patch.stop_after: - process_stop(stitch_plan) - - # process_stop() may have split the block into two - color_block = stitch_plan.last_color_block + color_block.add_stitch(stop=True) + color_block = stitch_plan.new_color_block(color_block.color) - add_ties(stitch_plan) + stitch_plan.filter_duplicate_stitches() + stitch_plan.add_ties() return stitch_plan @@ -64,6 +65,14 @@ class StitchPlan(object): def add_color_block(self, color_block): self.color_blocks.append(color_block) + def filter_duplicate_stitches(self): + for color_block in self: + color_block.filter_duplicate_stitches() + + def add_ties(self): + # see ties.py + add_ties(self) + def __iter__(self): return iter(self.color_blocks) @@ -198,12 +207,12 @@ class ColorBlock(object): stitches = [self.stitches[0]] for stitch in self.stitches[1:]: - if stitches[-1].jump or stitch.stop or stitch.trim: - # Don't consider jumps, stops, or trims as candidates for filtering + if stitches[-1].jump or stitch.stop or stitch.trim or stitch.color_change: + # Don't consider jumps, stops, color changes, or trims as candidates for filtering pass else: l = (stitch - stitches[-1]).length() - if l <= 0.1: + if l <= 0.1 * PIXELS_PER_MM: # duplicate stitch, skip this one continue @@ -247,11 +256,3 @@ class ColorBlock(object): maxy = max(stitch.y for stitch in self) return minx, miny, maxx, maxy - - def split_at(self, index): - """Split this color block into two at the specified stitch index""" - - new_color_block = ColorBlock(self.color, self.stitches[index:]) - del self.stitches[index:] - - return new_color_block diff --git a/lib/stitch_plan/stop.py b/lib/stitch_plan/stop.py deleted file mode 100644 index 0ccaeaf8..00000000 --- a/lib/stitch_plan/stop.py +++ /dev/null @@ -1,88 +0,0 @@ -from copy import copy - -from ..svg import PIXELS_PER_MM - - -def process_stop(stitch_plan): - """Handle the "stop after" checkbox. - - The user wants the machine to pause after this patch. This can - be useful for applique and similar on multi-needle machines that - normally would not stop between colors. - - In most machine embroidery file formats, there's no such thing as - an actual "STOP" instruction. All that exists is a "color change" - command. - - On multi-needle machines, the user assigns needles to the colors in - the design before starting stitching. C01, C02, etc are the normal - needles, but C00 is special. For a block of stitches assigned - to C00, the machine will continue sewing with the last color it - had and pause after it completes the C00 block. Machines that don't - call it C00 still have a similar concept. - - We'll add a STOP instruction at the end of this color block. - Unfortunately, we have a bit of a catch-22: the user needs to set - C00 (or equivalent) for the _start_ of this block to get the - machine to stop at the end of this block. That means it will use - the previous color, which isn't the right color at all! - - For the first STOP in a given thread color, we'll need to - introduce an extra color change. The user can then set the correct - color for the first section and C00 for the second, resulting in - a stop where we want it. - - We'll try to find a logical place to split the color block, like - a TRIM or a really long stitch. Failing that, we'll just split - it in half. - """ - - color_block = stitch_plan.last_color_block - - if not color_block or len(color_block) < 2: - return - - last_stitch = color_block.last_stitch - color_block.add_stitch(stop=True) - - if len(stitch_plan) > 1: - # if this isn't the first stop in this color, then we're done - if stitch_plan.color_blocks[-2].stop_after and \ - stitch_plan.color_blocks[-2].color == color_block.color: - return - - # We need to split this color block. Pick the last TRIM or - # the last long stitch (probably between distant patches). - - for i in xrange(len(color_block) - 2, -1, -1): - stitch = color_block.stitches[i] - - if stitch.trim: - # ignore the trim right before the stop we just added - if i < len(color_block) - 2: - # split after the trim - i = i + 1 - break - - if i > 0: - next_stitch = color_block.stitches[i + 1] - - if (stitch - next_stitch).length() > 20 * PIXELS_PER_MM: - break - - if i == 0: - # Darn, we didn't find a TRIM or long stitch. Just chop the - # block in half. - i = len(color_block) / 2 - - new_color_block = color_block.split_at(i) - - # If we're splitting in the middle of a run of stitches, we don't - # want a gap to appear in the preview and the PDF printout, so - # add an extra stitch to bridge the gap. Technically this will - # result in a double needle penetration but it's no big deal. - if not color_block.last_stitch.trim: - color_block.add_stitch(copy(new_color_block.stitches[0])) - - color_block.add_stitch(color_change=True, fake_color_change=True) - stitch_plan.add_color_block(new_color_block) -- cgit v1.3.1 From e0cecd6fa444e3b7ea2391ce4f2953ae9fd5f3fa Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Mon, 23 Jul 2018 20:15:49 -0400 Subject: fix a couple crashes --- lib/stitch_plan/stitch_plan.py | 4 ++++ lib/svg/svg.py | 3 +++ 2 files changed, 7 insertions(+) (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index 0fa87d71..a7cd60e8 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -45,6 +45,10 @@ def patches_to_stitch_plan(patches, collapse_len=3.0 * PIXELS_PER_MM): color_block.add_stitch(stop=True) color_block = stitch_plan.new_color_block(color_block.color) + if len(color_block) == 0: + # last block ended in a stop, so now we have an empty block + del stitch_plan.color_blocks[-1] + stitch_plan.filter_duplicate_stitches() stitch_plan.add_ties() diff --git a/lib/svg/svg.py b/lib/svg/svg.py index 5552abd8..48b1343a 100644 --- a/lib/svg/svg.py +++ b/lib/svg/svg.py @@ -37,6 +37,9 @@ def color_block_to_realistic_stitches(color_block, svg): paths = [] for point_list in color_block_to_point_lists(color_block): + if not point_list: + continue + color = color_block.color.visible_on_white.darker.to_hex_str() start = point_list[0] for point in point_list[1:]: -- cgit v1.3.1 From 1bd7aa110a1b30a6c44f4d792b3c817e10234c07 Mon Sep 17 00:00:00 2001 From: Lex Neva Date: Mon, 23 Jul 2018 20:22:53 -0400 Subject: change '# stops' in block to be 'stop after?' --- lib/stitch_plan/stitch_plan.py | 8 +------ messages.po | 37 ++++++++++++++++++++---------- print/templates/color_swatch.html | 4 ++-- print/templates/operator_detailedview.html | 12 ++++------ print/templates/operator_overview.html | 4 ++-- print/templates/print_overview.html | 4 ++-- 6 files changed, 36 insertions(+), 33 deletions(-) (limited to 'lib/stitch_plan') diff --git a/lib/stitch_plan/stitch_plan.py b/lib/stitch_plan/stitch_plan.py index a7cd60e8..682ea09f 100644 --- a/lib/stitch_plan/stitch_plan.py +++ b/lib/stitch_plan/stitch_plan.py @@ -97,7 +97,7 @@ class StitchPlan(object): @property def num_stops(self): - return sum(block.num_stops for block in self) + return sum(1 for block in self if block.stop_after) @property def num_trims(self): @@ -185,12 +185,6 @@ class ColorBlock(object): """Number of stitches in this color block.""" return len(self.stitches) - @property - def num_stops(self): - """Number of pauses in this color block.""" - - return sum(1 for stitch in self if stitch.stop) - @property def num_trims(self): """Number of trims in this color block.""" diff --git a/messages.po b/messages.po index 41a04cb3..19400443 100644 --- a/messages.po +++ b/messages.po @@ -8,7 +8,7 @@ msgid "" msgstr "" "Project-Id-Version: PROJECT VERSION\n" "Report-Msgid-Bugs-To: EMAIL@ADDRESS\n" -"POT-Creation-Date: 2018-07-24 20:19-0400\n" +"POT-Creation-Date: 2018-07-25 21:18-0400\n" "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" "Last-Translator: FULL NAME \n" "Language-Team: LANGUAGE \n" @@ -418,7 +418,7 @@ msgstr "" msgid "Embroidery Simulation" msgstr "" -#: lib/extensions/zip.py:66 +#: lib/extensions/zip.py:52 msgid "No embroidery file formats selected." msgstr "" @@ -434,7 +434,7 @@ msgid "" "file to lexelby@github." msgstr "" -#: lib/svg/svg.py:87 +#: lib/svg/svg.py:90 msgid "Stitch Plan" msgstr "" @@ -462,18 +462,27 @@ msgid "thread" msgstr "" #: print/templates/color_swatch.html:19 print/templates/color_swatch.html:43 -#: print/templates/operator_detailedview.html:64 +#: print/templates/operator_detailedview.html:62 msgid "# stitches" msgstr "" #: print/templates/color_swatch.html:23 print/templates/color_swatch.html:44 -#: print/templates/operator_detailedview.html:67 -msgid "# stops" +msgid "# trims" msgstr "" #: print/templates/color_swatch.html:24 print/templates/color_swatch.html:45 -#: print/templates/operator_detailedview.html:68 -msgid "# trims" +#: print/templates/operator_detailedview.html:66 +msgid "stop after?" +msgstr "" + +#: print/templates/color_swatch.html:24 print/templates/color_swatch.html:45 +#: print/templates/operator_detailedview.html:66 +msgid "yes" +msgstr "" + +#: print/templates/color_swatch.html:24 print/templates/color_swatch.html:45 +#: print/templates/operator_detailedview.html:66 +msgid "no" msgstr "" #: print/templates/footer.html:2 @@ -554,20 +563,24 @@ msgstr "" #: print/templates/operator_detailedview.html:33 #: print/templates/operator_overview.html:8 #: print/templates/print_overview.html:8 -msgid "Total nr stops" +msgid "Total stops" msgstr "" #: print/templates/operator_detailedview.html:34 #: print/templates/operator_overview.html:9 #: print/templates/print_overview.html:9 -msgid "Total nr trims" +msgid "Total trims" msgstr "" -#: print/templates/operator_detailedview.html:63 +#: print/templates/operator_detailedview.html:61 msgid "thread used" msgstr "" -#: print/templates/operator_detailedview.html:71 +#: print/templates/operator_detailedview.html:65 +msgid "trims" +msgstr "" + +#: print/templates/operator_detailedview.html:69 msgid "Enter operator notes..." msgstr "" diff --git a/print/templates/color_swatch.html b/print/templates/color_swatch.html index 6785b080..71022f6f 100644 --- a/print/templates/color_swatch.html +++ b/print/templates/color_swatch.html @@ -20,8 +20,8 @@ {% endif %} {# We don't want to see stops and trims if we have more than 13 colorSwatches to show #} {% if color_blocks|length < 13 %} - {{ _('# stops') }}: {{ color_block.num_stops }} {{ _('# trims') }}: {{ color_block.num_trims }} + {{ _('stop after?') }}: {{ _("yes") if color_block.stop_after else _("no") }} {% endif %} @@ -41,8 +41,8 @@

{{ _('rgb') }}:{{ color_block.color.rgb }}

{{ _('thread') }}:{{ color_block.color.manufacturer }} {{ "#" + color_block.color.number if color_block.color.number }}

{{ _('# stitches') }}:{{ color_block.num_stitches }}

-

{{ _('# stops') }}:{{ color_block.num_stops }}

{{ _('# trims') }}:{{ color_block.num_trims }}

+

{{ _('stop after?') }}:{{ _("yes") if color_block.stop_after else _("no") }}

diff --git a/print/templates/operator_detailedview.html b/print/templates/operator_detailedview.html index e1899db4..e3c0d441 100644 --- a/print/templates/operator_detailedview.html +++ b/print/templates/operator_detailedview.html @@ -30,18 +30,16 @@ {{ _('Total stitch count') }}: {{job.num_stitches }}

- {{ _('Total nr stops') }}: {{ job.num_stops }} - {{ _('Total nr trims') }}: {{ job.num_trims }} + {{ _('Total stops') }}: {{ job.num_stops }} + {{ _('Total trims') }}: {{ job.num_trims }}

- {% for color_block in color_blocks %} -

@@ -64,13 +62,11 @@ {{ _('# stitches') }}: {{ color_block.num_stitches }}

- {{ _('# stops') }}: {{ color_block.num_stops }} - {{ _('# trims') }}: {{ color_block.num_trims }} + {{ _('trims') }}: {{ color_block.num_trims }} + {{ _('stop after?') }}: {{ _("yes") if color_block.stop_after else _("no") }}

{% endfor %} - - diff --git a/print/templates/operator_overview.html b/print/templates/operator_overview.html index 0c379316..048fd58e 100644 --- a/print/templates/operator_overview.html +++ b/print/templates/operator_overview.html @@ -5,8 +5,8 @@

{{ _('Unique Colors') }}:{{ job.num_colors }}

{{ _('Color Blocks') }}:{{ job.num_color_blocks }}

-

{{ _('Total nr stops') }}:{{ job.num_stops }}

-

{{ _('Total nr trims') }}:{{ job.num_trims }}

+

{{ _('Total stops') }}:{{ job.num_stops }}

+

{{ _('Total trims') }}:{{ job.num_trims }}

diff --git a/print/templates/print_overview.html b/print/templates/print_overview.html index ca8638ff..cfbb81d1 100644 --- a/print/templates/print_overview.html +++ b/print/templates/print_overview.html @@ -5,8 +5,8 @@

{{ _('Unique Colors') }}:{{ job.num_colors }}

{{ _('Color Blocks') }}:{{ job.num_color_blocks }}

-

{{ _('Total nr stops') }}:{{ job.num_stops }}

-

{{ _('Total nr trims') }}:{{ job.num_trims }}

+

{{ _('Total stops') }}:{{ job.num_stops }}

+

{{ _('Total trims') }}:{{ job.num_trims }}

-- cgit v1.3.1