Bug: issue with Intersect Arcs tool

Discuss issue with Intersect Arcs tool here, separate from issue with Intersect Curves tool. Thanks!

image
image

1 Like

image
image

1 Like

image

1 Like

But I WAS using the intersectcurves tool. You can use arcs with the curve tools. And if you note the last example I gave it was an intersect of an Arc and a curve.

1 Like

Is this the right thing, that the Intersect Curves tool accepts two arcs? Hmmm.

1 Like

Yes. Thatā€™s my point. There are some inherent design flaws with the existing tools.

2 Likes

Itā€™s actually a really good point about the ā€œintersect curvesā€ tool accepting two arcs. That leads to some interesting corner cases and since there is an ā€œintersect arcsā€ tool, perhaps itā€™s worth just not allowing it.

Problem of course is that if people already have pattern using this feature then it then removing it will break their patternsā€¦

2 Likes

@rode_kater & @Douglas

What do you think about the following:

Intersect Curve tool:

  • Restrict to Splines and SplinePaths as input
  • Create curve segments for each side of the intersection
  • Intersections with endpoints are allowed
  • Allow a maximum of 4 intersection points
  • Update Options to select 1st point of intersection, 2nd point of intersection, 3rd point of intersection, 4th point of intersection.

Intersect Arcs tool:

  • Same as Intersect Curve tool, exceptā€¦
    • There will be a max of 2 intersection points unless:
      • the arcs are exactly the same (infinite intersection points), or
      • one is a subset of the other (infinite intersection points)
      • Question: Should the intersection defaults to:
        • the first & last points of the shared arc, or
        • the origin point like our other tools

Intersect Curve and Arc tool:

  • Same as Intersect Curve tool
2 Likes

Hate to throw another wrench in the works - Donā€™t forget that an Elliptical arc can also be selected in using the Point intersection curves & Point intersect curve & axis tools. <sigh>

Note: The Point intersect curve & axis and Point intersect arc & axis use the same tool code. Now by separating the curves & arcs (and ellipses) youā€™re not talking about doubling or tripling THAT tool.

I need to give it some thought and check a few things, but rememberā€¦ changing any of the tools by adding or deleting properties will likely result in updating the schema and could result in breaking existing patterns requiring conversion from the old to new. I can only assume users have used these intersect tools so it could be a big headache.

2 Likes

W.r.t. the problem of arcs intersecting multiple times, Iā€™m not sure if this is something we need to worry about. How often is this likely to come up in a real pattern?

Also, the option of letting people choose ā€œ1st, 2nd, etcā€ doesnā€™t sound like something useful in pattern making. Instead youā€™d probably want something like ā€œthe intersection between points A & B (which are on the curve already)ā€. Besides, which is the ā€œfirstā€ intersection kinda depends on your point of view. Similarly highest/lowest/leftmost/rightmost (like now) seem also to be useful for pattern maker, weā€™re not making a mathematical geometry program (I think).

You achieve almost the same effect by allowing people to split curves into sub-curves, and then using the sub-curves as input to the tools: same effect, different method.

So I think what is there now is almost good, except I think forbidding selecting two arcs is good (though perhaps you could automatically convert it to the other tool?).

2 Likes

Iā€™m having an issue with this at the moment. If I make 1 arc too short in diameter, it wonā€™t take the shoulder length when I resize the pattern to a smaller sizes, but if I make the arc longer, then I get the arcs intersecting in 2 places on the larger sizes. So the intersection only works on the mid sizes. So yes, it does occur in a real pattern. At the moment, Iā€™ve put in a formula to widen the shoulder dart to get my pattern to work on a smaller arc diameter. Since the dart gets closed up on this particular pattern, itā€™s possible to use this solution. :slight_smile:

1 Like

counting from the origin point of the first curve selected, probably.

2 would be quite common. 3 or moreā€¦ yeah, Iā€™m not so sure about thatā€¦ idea! A wavy coat closure. I donā€™t have time to pursue that idea further tonight, but a rough ā€œsketchā€ indicates that Iā€™m probably right. It could call for a dozen! (or perhaps I just donā€™t understand pattern-making well enough.)

1 Like

One of the problems with discussing this is how hard it is to differentiate ā€œIntersect Arcsā€ & ā€œIntersect Curvesā€. As it stands at the moment, I feel like weā€™d be best off with two buttons for the same tool, but thatā€™s just the feels of the moment.

So hereā€™s my example of multiple intersections. Pardon my obsession with symmetry, it was not worth fighting this day. :crazy_face:

WibblyClosure.val (4.4 KB)

WibblyClosure

As you can see, there are six intersections in just this one little clip. Although I was envisioning using the intersections along the axis, which it would be possible to point in a couple of different ways, I think the left or right intersections might actually be better.

1 Like

Yes, I think this one is a toughie. Iā€™d suggest keeping things as simple as possible and to revert back to using the intersection of curve & axis tool here to get the intersections. It would mean more nodes.

Is it at all possible to put all of the curve/arc tools into the curve tools (since they are all effectively curves) so that all the tools under the curves will work on arcs & ellipses, as well as the curves? It seems to me that the only difference is how the curve is originally made that is different and that arc/ellipse do not have start and end nodes - so the segment tool wonā€™t work on them.

If it were up to me, Iā€™d even go so far as to put the arc & ellipse into the curve section so that everything is under the curves. Could this reduce the programming down to only 1 tool per type for all three and add some space to the tool section?

I really do like the 1st option, as a program user, however, the 2nd option has made me re-examine shoulder darts to correct my patterns for people with larger bust sizes and narrower shoulders or vice versa :grin:

Anyway, this is my few pennies worth. Thank you very much for all the thought and programming you are all putting into this amazing software. :star_struck: :star_struck: :star_struck:

1 Like

@ Douglas Understood. Yet the tool we have is not robust. There are 24 intersection permutations, and these map to 18 distinct algorithms. (see intersection combo list below)

We could overload the current intersect function so that based on the parameter combo one of 18 functions with the same name will perform the calculations specifically for that combo. This way the logic can be fine-tuned and it doesnā€™t break current pattern files. This approach is fast and a lot of people use it in the c++ world. Function overloading in C++

Another approach is to use a case statement in our current function. The case statement tests for the 18 combos then passes control to functions that are named for that specific combo, like ā€œintersectSplArc.ā€ This code might be easier to maintain because of the readability of function names plus the passing of control is obvious. This also doesnā€™t break current pattern files.

Intersections ( 6 each for Spl, SplPath, Arc, Ellipse, 6 x4 = 24. There are 6 duplicates, 24 - 6 = 18. Including ā€˜Lineā€™ because weā€™ve had user requests for this, so nowā€™s the time to add this.):

  1. Spl + Line
  2. Spl + Axis
  3. Spl + Spl
  4. Spl + SplPath
  5. Spl + Arc
  6. Spl + Ellipse
  7. SplPath + Line
  8. SplPath + Axis
  9. SplPath + Spl (same as #4 Spl + SplPath)
  10. SplPath + SplPath
  11. SplPath + Arc
  12. SplPath + Ellipse
  13. Arc + Line
  14. Arc + Axis
  15. Arc + Spl (Same as #5 Spl + Arc)
  16. Arc + SplPath (Same as #11 SplPath + Arc)
  17. Arc + Arc
  18. Arc + Ellipse
  19. Ellipse + Arc
  20. Ellipse + Line
  21. Ellipse + Spl (Same as #6 Spl + Ellipse)
  22. Ellipse + SplPath (Same as #12 SplPath + Ellipse)
  23. Ellipse + Arc (Same as #18 Arc + Ellipse)
  24. Ellipse + Ellipse
2 Likes