# Full Version: Planar Holes Finder : a script for 2D workflows!

From: mkdm [#12]
25 Oct 2017
To: ALL

Ciao Mik!

Thanks a lot for your appreciation! Thanks!

Your idea to combine this script with your "SelectInetrescted" it's a great idea!

I hope soon, maybe tonight, to release the 0.2 version of "PlanarHolesFinder" and also the special modified version "LassoSelCurves"
and the in the next days I will study "SelectInetrescted" to see what can I do to combine both!

It could be a great thing having these kind of scripts combined, but also in the case it will be too complex,
with the very fast workflow of Moi's shortcut we can always fire commands/scripts at the speed of light :)

My intention is to create a little set of very specialized script especially dedicated to 2D Workflows,
and "PlanarHolesFinder"/"LassoSelCurves" are just the first,
but...at the moment it's best for me not make too many plans :)

Stay tuned!

Ciao!

Marco (mkdm)

From: Michael Gibson [#13]
25 Oct 2017
To: ALL

Hi Marco,

> But...I didn't understand very well this part of the sentence :
>
> @You : "It doesn't work too well with curvy surfaces when the curve isn't right near the surface though."

If you have a curvy surface instead of a flat surface, this type of projection down by surface normal will make the projected result squish together or expand and the Solids++ routine for this function doesn't deal very well with that, it tends to produce a curve with little zig-zags in it. If the curve is pretty close to a wavy surface it can work ok though.

But I need to make a new implementation for it at some point here since it can give bad results.

- Michael

From: mkdm [#14]
26 Oct 2017
To: ALL

Hi Michael.

Ok. Understood.

@You : "...If you have a curvy surface instead of a flat surface..."
But...the problem is that I never have Non-flat surface.
The surface that I use with as the target for "Project" is always flat, but there are some situation where it fails anyway.

Strange thing...

You can watch again my previous post and the video posted at (http://moi3d.com/forum/index.php?webtag=MOI&msg=8646.7)

This is the video : http://take.ms/AF6Rq

As you can see at position 01:58 I use the Circle as reference to find holes in it.
The circle if is simply rotated but is FLAT.
And as you can see the blue curve is not considered "hole". Is not "captured" by the "Project" command.

I stay tuned.

Thanks.

From: mkdm [#15]
26 Oct 2017
To: ALL

...maybe I should switch to "Direction" mode for the "Project" command.

But I wanted to use "ClosestPt" because it's more easy and requires less interaction with the user, less inputs from him.

I think if I switch to "Direction" for "Project", if I want to keep the code simple at least for this first upcoming official release of the script,
my command will work only for Top Plane.

Instead using "ClosestPt" I've seen that it works also for all standard planes (right, left....)

I don't know.

Also I'm writing this script to be placed in "Scripts" folder, non in "Commands" because I don't want for it any UI.
Just fire a shortcut and run it, both Lasso and "FindHoles" variants.

What do you think ?

Thanks.

From: Michael Gibson [#16]
26 Oct 2017
To: ALL

Hi Marco, yes it sounds like the "closest point" mode for project isn't reliable enough for you, the direction mode should work better. Or another possibility could be the steps that I mentioned earlier - determine if any curves intersect the outer boundary (use the "intersect" factory from the Construct > Curve > Isect command and see if it generates any result), remove those then send the rest to planarsrf to generate loops. Use the hole loops to identify the curves they came from by seeing if any start/end points match between the curve segments and the hole edges.

- Michael

From: mkdm [#17]
26 Oct 2017
To: ALL

Hi Michael.

Thanks a lot for the suggestion.

But...I think that the method you mentioned is not right for me.

Because this part of the sentence " if any start/end points match between the curve segments and the hole edges" it's too restrictive.
I could have more than one curve that shares the same start/end point but with different shapes.

For example :

Furthermore also this part of the sentence is not what I want to do : "...remove those then send the rest to planarsrf to generate loops."

Because I want to use my script to find not only "totally enclosed curves" but also "partially enclosed", that is the curves that intersect the boundary of
the "container".

I really don't know.

Maybe for the moment I can continue to use "Project" with "ClosestPt" and for the moment not to take into account the rare cases where it fails .

Or I can decide to witch to "Direction" mode and for the moment restrict the scope of my scripts to "TOP plane only", that also was the original idea.

I don't know.

Have you got a final suggestion to give me about these considerations ?

Thanks a lot :)

Marco (mkdm)

From: Michael Gibson [#18]
26 Oct 2017
To: ALL

Hi Marco, my suggestion for the shared point case you show there would be to not consider that to be a hole, that will be malformed geometry if you try to actually create a hole like that where a hole boundary intersects with the outer boundary. Boundaries should not be self intersecting and outer/inner boundaries should not intersect with each other. If they do then there become areas of a trimmed surface where the inside and outside regions of the surface are not well defined.

For the other part you mention:

> Furthermore also this part of the sentence is not what I want to do : "...remove those then send the rest to planarsrf to generate loops."
>
> Because I want to use my script to find not only "totally enclosed curves" but also "partially enclosed", that is the curves
> that intersect the boundary of the "container".

Ok, then instead of removing the intersected curves from consideration just put them also into the "captured" list but don't send them over to planarsrf.

But another problem is it sounds like you're not just looking for "holes" anymore, to make a "hole" it has to be a closed curve.

- Michael

From: Michael Gibson [#19]
26 Oct 2017
To: ALL

Hi Marco, also for direction I think you should be able to automatically figure out a projection direction by getting the bounding box of your containment curve and then seeing which direction has a zero extent.

- Michael

From: mkdm [#20]
26 Oct 2017
To: ALL

@You : "...But another problem is it sounds like you're not just looking for "holes" anymore, to make a "hole" it has to be a closed curve..."

You're right Michael :)

The fact is that I wanted initially to start a script for "2D workflows" not for "solids" or "solids boolean" stuff.

My script does this main things :

1) Capture all curves that can be considered "holes" of another flat curve.
I know that this part of the sentence was not very clear.
When Is said "Hole" I wanted to say : closed curves that, when projected, are totally enclosed into the "planar surface" created using the"container" curve.

2) During my coding I also wanted to add capabilities to capture also other situations.
So, now I'm coding my script to be fired in 3 different ways :

A) Capture all "totally enclosed" in "container curve"
B) Capture all "partially enclosed" in "container curve", that is the curves that when projected they intersect the boundary of the "container"
3) Capture both of them

But...during the coding and also during the various experiments and tests popped into my mind that the scripts also could works with
other planes, not only TOP plane, and I started to face with the problem of "Direction" vs "ClosestPt."

Now I try to switch to "Direction" mode and leave the script with the minimum interaction with the user,
assuming that the direction in forced to TOP plane.

Later, when the first official version of the script will be tested by other users, I will try to make it more complex.

I'm also writing the variant for a quick "Lasso" selection !

Thanks for all your help :)

I think I need some suggestion for using the "Project" factory with "Direction", but I want first to try.

We'll catch up!

Ciao!

Marco (mkdm)

From: Michael Gibson [#21]
26 Oct 2017
To: ALL

Hi Marco, we might have been writing posts at the same time so make sure not to miss my additional message above about how to automatically determine the projection direction.

Also another thing you might be able to use for containment detection is Construct > Curve > Isect - if you build a planarsrf with your main boundary you could then run intersect on other curves and it will return points if the curve is on the face.

- Michael

From: Michael Gibson [#22]
26 Oct 2017
To: ALL

Also project factory inputs are like this:

0: ObjectList - BaseObjects
1: ObjectList - TargetObjects
2: String - Mode
3: Point - Direction start pt
4: Point - Direction end pt
5: Bool - Delete inputs
6: CoordinateFrame - Projection plane - When set project to this plane rather than onto target objects.

- Michael

From: mkdm [#23]
26 Oct 2017
To: ALL

Michael thanks you very much for your precious help! Much appreciated !!

Well...I hold in the highest regards all your suggestions and I keep them into my mind.

But for the moment I decided to publish the very first official version of this script using "closestpt" mode of Project command.

I have seen that, apart some rare cases, it gives me much more flexibility and I can work on all views.

In a few hours I will post the first version 1.0 beta also with the variant for lasso selection.

I hope that many users will try these scripts and will report me all issues or strange things.

In this case I will switch to a more complex "curve detection" using "Direction" mode or other ways.

For the moment I think and hope that this couple of scripts, including "Lasso" selection, will be very useful for many 2D workflows and
for other Moi's use cases in general.

Just a final question : Do you think I should create a brand new thread for this first official release to give it the chance to be viewed and tried by
as many users as possible ?

I don't want to spam this forum with all my things ... :)

Thanks again.

Wel'll catch up!

Ciao.

From: mkdm [#24]
26 Oct 2017
To: ALL

P.S. ...

Meanwhile could you please remove the old thread that I created some days ago named : "A possible script for finding "holes" of a planar closed curve" ?

Thanks a lot.

Message 8646.25 was deleted

From: Michael Gibson [#26]
26 Oct 2017
To: ALL

Hi Marco,

re:
> Do you think I should create a brand new thread for this first official release to give
> it the chance to be viewed and tried by as many users as possible ?

Sure, that sounds like a good idea.

re:
> Meanwhile could you please remove the old thread that I created some days ago named : "A
> possible script for finding "holes" of a planar closed curve" ?

Are you sure you need to have it removed? Since there was some discussion there about possible strategies maybe it could be useful for someone doing research on how to approach a particular kind of script in the future...

- Michael

From: mkdm [#27]
26 Oct 2017
To: ALL

Hello Michael.

Ok. I will publish this first beta version of these two scripts in a new thread.

I'm doing some final test and I hope that in a couple of hours, starting now, I should be done.

Unfortunately, like you said, during the last tests the "ClosePt" method for the "Project" command is proving to be too unreliable in many cases.

But I want to close this first beta soon, so i think I should switch to "Direction" mode and calculate the direction based on the bounding box of the container curve.

This will take at least one hour to make some test.

@You : "...Are you sure you need to have it removed? Since there was some discussion there about possible strategies..."

I asked you to remove the other thread because I thought that all the most important things have been addressed here
and all the 5 posts there are only from you and me. No other users.

If you want you can remove it.

Now I go to make the final coding and test.

We'll catch up!