[Plugin] Point Merger tools v1.2 UPDATED@ 23/06/09

JClements

The script does not quite work for me like what is shown in the video.

Perhaps I copied the wrong files to the Plugin directory?

From the ZIP folder I copied the two .rb's shown below:

Chris Fullmer

Hi John, I think you just need to copy those 2 .rb files into your plugins folder. They don't need to be in the subfolder - unless I'm missing something. I'll edit that out if its wrong.

@BTM - The reason that the class variables are not working correctly (meaning they are not keeping their value during the entire SU session) is because you are declaring the variable in your activate method. So everytime the activate the tool gets activated, it resets the @@max_weld_distance variable back to 0.0.to_l.

So what you want to do is declare that variable inside the class, outside of the methods. Like this:

class MergePoints @@max_weld_distance=0.0.to_l

I know it works, and I think that is the correct way to do it. it feels weird to have code not wrapped up in a method....

BTM

@chris fullmer said:

Hi John, I think you just need to copy those 2 .rb files into your plugins folder. They don't need to be in the subfolder - unless I'm missing something. I'll edit that out if its wrong.

Chris, you're right, they both go into the plugins folder.

@chris fullmer said:

@BTM - The reason that the class variables are not working correctly (meaning they are not keeping their value during the entire SU session) is because you are declaring the variable in your activate method. So everytime the activate the tool gets activated, it resets the @@max_weld_distance variable back to 0.0.to_l.

I know that the variable needs to be outside of a method, but with things like
@@max_weld_distance=1.to_l if not @@max_weld_distance
it would call it an uninitialized variable, and give an error message.

@chris fullmer said:

So what you want to do is declare that variable inside the class, outside of the methods. Like this:

class MergePoints @@max_weld_distance=0.0.to_l

It does work! I've been trying if functions, (so that if @@max_wld_distance doesn't have a value, it gets set to 0.0.to_l), but I kept getting error messages, and I guess that an if function isn't necessary.

JClements

OK,

I guess my original download of the two .rb's DID work. It was the UI that threw me off.

When I selected the Merge Points submenu option, I expected the Dialog box to appear, but none did.

1)
Only by accident did I discover I had to hit the [Enter] key to display it.

Question, shouldn't the dialog appear as soon as you select Merge Points from the submenu?

Also, if that dialog or the Taskbar could include a message to "press [Enter] to change value" that would be helpful (if you don't use the script frequently, then it may take a bit of experimentation to use it properly )

2)
Regarding Merge Points to Line, it seems to do just the opposite. The dialog box appears when you first select it from the submenu, but you can't recall the dialog by pressing [Enter], you have to go back to the submenu. Can [Enter] be used to recall the appropriate dialog too?

3)
Once used Undo doesn't work well. Is there a way to undo all changes performed between mouse-clicks at once?

BTM

@jclements said:

OK,

I guess my original download of the two .rb's DID work. It was the UI that threw me off.

When I selected the Merge Points submenu option, I expected the Dialog box to appear, but none did.

1)
Only by accident did I discover I had to hit the [Enter] key to display it.

Question, shouldn't the dialog appear as soon as you select Merge Points from the submenu?

Also, if that dialog or the Taskbar could include a message to "press [Enter] to change value" that would be helpful (if you don't use the script frequently, then it may take a bit of experimentation to use it properly )

2)
Regarding Merge Points to Line, it seems to do just the opposite. The dialog box appears when you first select it from the submenu, but you can't recall the dialog by pressing [Enter], you have to go back to the submenu. Can [Enter] be used to recall the appropriate dialog too?

3)
Once used Undo doesn't work well. Is there a way to undo all changes performed between mouse-clicks at once?

OK, I'll address some of these issues soon. I'm adding "Press Enter/ Return to set radius" in the status box for the Merge Points tool in the next release. As for the dialogue appearing as soon as you select the tool, I find it easier to use it like a normal tool with a radius, such as the smoove tool does. I want to get the VCB working later too, then it would save a lot of confusion, but that seems a bit tricky to do.

As for the 'Merge Points to Lines' tool, it affects the model upon activation, so I don't see why you'd need to open it again.

…Lastly, I think there's a method I could find somewheres that would make all the transformations into one action for undo, but I'm not sure. I'll look into it.

BTM

Here's a little video of me trying to use the 'Merge Points to Lines' tool to do some stitching with GE terrains.
http://www.screencast.com/t/s1SkJiUjH
It's 3 minutes long, because I kept changing styles, softening, etc., and I couldn't use my toolbar due to the size of the window.

steps:

• explode, unhide and unsoften 2 snapshots from google earth.

• figure out what the max range for the tool should be. (I use line tool to see how much things would have to move, then make the max about 25% bigger than that, just in case.)

• completely select one of the meshes.

• use the 'Merge Points to Lines' tool.

• Select both meshes (because sketchup doesn't connect the points together automatically for some reason), and use intersect. Select the mesh, soften it, and fix any problem areas(there shouldn't be too many)

• Repeat these steps to add other snapshots to the previous mesh.

CameronVan

Thanks for this script. It will prove very handy.

chango70

Great script!

I just tested it and 2 things came up.

I noticed that when I press undo after a point Merge operation it does not undo the entire procedure, rather breaks it down to individual line moves.
I cannot get it to work if the geometry is in side a group.

If those 2 issue can be addressed this would one of the most useful scripts out there! Well deserve a custom button I think, hint hint CADFather

chango70

Oh yes one more thing. Would be great if you can add a CTLR modifier to make the new mesh smooth. Oh I am just been greedy now. Great job !

BTM

@chango70 said:

I noticed that when I press undo after a point Merge operation it does not undo the entire procedure, rather breaks it down to individual line moves.

I've got this working properly now, and it will be fixed in the next release for the tools.

@chango70 said:

I cannot get it to work if the geometry is inside a group.

I'll try to get that fixed too, but maybe not for the next release.

@chango70 said:

Would be great if you can add a CTLR modifier to make the new mesh smooth.

…I probably won't add this, but if I do decided to, it won't be anytime soon.

Also as a side note, I'm planning on adding a toolbar for them, and am trying to figure out how to add a subfolder and a loader script. I've got the toolbar working, but not with a subfolder or loader, and having the files just sitting in the plugins folder is a bit annoying.

CadFather

BTM

@cadfather said:

Nice!
I'll see if I can make one almost as good for Merge Points to Lines

chango70

@cadfather said:

:roll:

[attachment=1:cjv1mmue]pmerge_16.png[/attachment:cjv1mmue][attachment=0:cjv1mmue]pmerge_24.png[/attachment:cjv1mmue]

You da man!

chango70

@unknownuser said:

@chris fullmer said:

Hi John, I think you just need to copy those 2 .rb files into your plugins folder. They don't need to be in the subfolder - unless I'm missing something. I'll edit that out if its wrong.

Chris, you're right, they both go into the plugins folder.

@chris fullmer said:

@BTM - The reason that the class variables are not working correctly (meaning they are not keeping their value during the entire SU session) is because you are declaring the variable in your activate method. So everytime the activate the tool gets activated, it resets the @@max_weld_distance variable back to 0.0.to_l.

I know that the variable needs to be outside of a method, but with things like
@@max_weld_distance=1.to_l if not @@max_weld_distance
it would call it an uninitialized variable, and give an error message.

@chris fullmer said:

So what you want to do is declare that variable inside the class, outside of the methods. Like this:

class MergePoints @@max_weld_distance=0.0.to_l

It does work! I've been trying if functions, (so that if @@max_wld_distance doesn't have a value, it gets set to 0.0.to_l), but I kept getting error messages, and I guess that an if function isn't necessary.

Hey wow that would be great!

chango70

@unknownuser said:

@chango70 said:

I noticed that when I press undo after a point Merge operation it does not undo the entire procedure, rather breaks it down to individual line moves.

I've got this working properly now, and it will be fixed in the next release for the tools.

@chango70 said:

I cannot get it to work if the geometry is inside a group.

I'll try to get that fixed too, but maybe not for the next release.

@chango70 said:

Would be great if you can add a CTLR modifier to make the new mesh smooth.

…I probably won't add this, but if I do decided to, it won't be anytime soon.

Also as a side note, I'm planning on adding a toolbar for them, and am trying to figure out how to add a subfolder and a loader script. I've got the toolbar working, but not with a subfolder or loader, and having the files just sitting in the plugins folder is a bit annoying.

Fantastic! I can't wait for the new version. Great work!

CadFather

..haven't tried the script yet so didn't notice there's two... coming soon....

BTM

…these are the icons it has right now, I'll wait to see how yours turn out

CadFather

i think these are very good BTM - no need of others..

BTM

…I also have these ones though…

…I'll go with your icons for 'Merge Points', but now for Merge Points to Selection (I changed the name, because it works when selecting faces now aswell), I don't know which one to pick

Oh, by the way, they work in groups and components now too. I should have put @model.active_entities instead of @model.entities

…I've also included a cursor for 'Merge Points'. It's ugly though, so I'm changing it, and I don't know what the new one should look like.

CadFather

looking good..