satin hatching does not catch all polygons about pembroider HOT 7 OPEN

fixed: 7099df3

btw, optimize shouldn't be used with SATIN, since it might flip the order of the centerline and the hatch, see #82

from pembroider.

@stephanschulz also, be aware that beginOptimize()/endOptimize() does exist, in case you need to combine SATIN with other stuff.

from pembroider.

Hi @stephanschulz , I'm wondering what is your input to SATIN? Is the input polygons vertices, or binary images? Is each color input separately to SATIN, or is each polygon input separately to SATIN, or are they given all at the same time? Do the missing parts share the same color, and of what color should they actually be? What does gray values of the second grayscale image indicate, do they correspond to the colors of the first image?

Thanks and sorry for so many questions, but I need the info to pinpoint the problem :)

from pembroider.

• input is a bunch of binary images
• each image has a different color
• the missing parts do share the same color but are also part of the same binary image
• the second image is a illustrator file to show you the different layers.

Embroider_test.zip

from pembroider.

import java.io.*;
import processing.embroider.*;
PEmbroiderGraphics E;


void setup(){
  size(800,800);
  background(0);
  
  E = new PEmbroiderGraphics(this);
  
  String[] filenames = new File(dataPath("")).list(new FilenameFilter() {
    public boolean accept(File dir, String name) {
        return name.toLowerCase().endsWith(".png");
    }
  });
  

  for (int i = 0; i < filenames.length; i++){
    println("working on layer "+i+" ...");
    PImage img = loadImage(dataPath(filenames[i]));
    img.resize(width,height);
    
    E.noStroke();
    E.fill((int)random(100,255),(int)random(100,255),(int)random(100,255));
    E.hatchSpacing(5);
    E.hatchMode(E.SATIN);
    E.image(img,0,0);

  }
  
  E.visualize(true,false,false);

}

from pembroider.

as soon as I add

  E.optimize();
  E.visualize(true, false, false);
  E.printStats();
  E.endDraw(); // write out the file

to your code I see the problem again

from pembroider.

thanks for the tips.

I saw E.optimize(); being used in the PEmbroider_satin_hatching_2 example. But now I know better.

thanks.

from pembroider.