New API doc - typos and questions

Chris Fullmer

The FAQ page lists some sites that carry/catalog ruby scripts. the Sketchucation extensions index (thanks to Jim!) would also be a great resource to add to that list now

extensions url:

410 Gone

(www.sketchucation.com)

Chris

thomthom

http://code.google.com/apis/sketchup/docs/ourdoc/toolsobserver.html
It'd be nice if the ToolsObserver listed the full list of tools IDs.

I've begun a list here which associates constants to the IDs for convenient use in scripts: http://www.sketchucation.com/forums/scf/viewtopic.php?f=180&t=18124

Jim

@thomthom said:

http://code.google.com/apis/sketchup/docs/ourdoc/toolsobserver.html
It'd be nice if the ToolsObserver listed the full list of tools IDs.

I've begun a list here which associates constants to the IDs for convenient use in scripts: http://www.sketchucation.com/forums/scf/viewtopic.php?f=180&t=18124

In the back of my mind I have the idea the the tool id's were not reliable and shouldn't be used; but I can't remember why.

thomthom

@jim said:

@thomthom said:

http://code.google.com/apis/sketchup/docs/ourdoc/toolsobserver.html
It'd be nice if the ToolsObserver listed the full list of tools IDs.

I've begun a list here which associates constants to the IDs for convenient use in scripts: http://www.sketchucation.com/forums/scf/viewtopic.php?f=180&t=18124

In the back of my mind I have the idea the the tool id's were not reliable and shouldn't be used; but I can't remember why.

I thought is was the tool names as on Mac the first four characters where truncated? No?

Chris Fullmer

http://code.google.com/apis/sketchup/docs/ourdoc/componentinstance.html#definition

The ComponentInstance.definition example does not include the method in the example anywhere, so by definition it is not really an example.

This is the given example:

 point = Geom;;Point3d.new 10,20,30
 transform = Geom;;Transformation.new point
 model = Sketchup.active_model
 entities = model.active_entities
 path=Sketchup.find_support_file "Bed.skp",
   "Components/Components Sampler/"

Jim

Any ideas about a way to mark these errors as they are fixed?

thomthom

We could ask Coen to have a strike-through BBC tag. [deleted] or something...
Or, put a big fat FIXED in bold green. Might be easier.

avariant

From Image.explode:

entitiesarray     an Array object of entities if successful

which I interpret as the face and edges created in the explosion. However, the result from my testing always returns an empty array.

thomthom

I see that behaviour as well: http://www.sketchucation.com/forums/scf/viewtopic.php?f=11&t=17150&hilit=+explode

Chris Fullmer

http://code.google.com/apis/sketchup/docs/ourdoc/polygonmesh.html#add_polygon

add polygon detail just repeats the add_point example.

scottlininger

@thomthom said:

Seems that InstanceObserver.onClose is dead as well. http://code.google.com/apis/sketchup/docs/ourdoc/instanceobserver.html#onClose
But onOpen works.

Thomas,

I'm about to push an update with all of the recent changes above. This one puzzled me, though. The ToolsObserver examples work fine for me on the onOpen and onClose. Can you clarify what "dead as well" means?

@unknownuser said:

Does this list work 'across-platforms' - my understanding was that PCs and Macs differ ?

Tig, these action codes only work on the PC, but I still think I'll add them to the docs. There are some cool things in there that PC-only shops might need.

Thanks, guys! New update should be live in the next day or so.

scottlininger

@thomthom said:

We could ask Coen to have a strike-through BBC tag. [deleted] or something...
Or, put a big fat FIXED in bold green. Might be easier.

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

thomthom

@unknownuser said:

I'm about to push an update with all of the recent changes above. This one puzzled me, though. The ToolsObserver examples work fine for me on the onOpen and onClose. Can you clarify what "dead as well" means?

Dead as in: I never see it trigger.

thomthom

ToolObservers? How are they related to onClose and onOpen?

remus

@unknownuser said:

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

I'll ask gai if he can make you a mod for the ruby forum

scottlininger

@thomthom said:

ToolObservers? How are they related to onClose and onOpen?

I meant to say, InstanceObservers.

In any case, they fire fine for me.

Jim

@remus said:

@unknownuser said:

Jim, is there a way that I could be granted edit rights just to this thread? Then I could add a FIXED note inside the reports as I correct them?

I'll ask gai if he can make you a mod for the ruby forum

Scott, there's no way to give edit-rights on a single topic. So you get edit rights to the whole Ruby Forum.

If you have a list of changes, we can take care of marking them fixed. I'd hate to see you spend time doing it if you have more important work to do.

Gaieus

Hi Scott, I have sent you a PM, too, about it. The mods have also be informed to welcome you in the team.

thomthom

@unknownuser said:

@thomthom said:

ToolObservers? How are they related to onClose and onOpen?

I meant to say, InstanceObservers.

In any case, they fire fine for me.

Hmm... When I tried to implement it in my plugin the onClose never fired. I also tried Jim's observer test script: http://sketchuptips.blogspot.com/2008/03/plugin-oberver-tests.html - and that never fired either. Could you post a working sample where onClose fires? (The onOpen always works though.)

scottlininger

@thomthom said:

Could you post a working sample where onClose fires?

Give this a go... Could be that we're seeing a sneaky bug...

class MyInstanceObserver < Sketchup;;InstanceObserver
   def onOpen(instance)
     UI.messagebox("onOpen; " + instance.to_s)
   end

   def onClose(instance)
     UI.messagebox("onClose; " + instance.to_s)
   end
end

model = Sketchup.active_model
model.definitions[0].instances[0].add_observer(MyInstanceObserver.new)