New API doc - typos and questions

thomthom

Vector3d.linear_combination
http://code.google.com/apis/sketchup/docs/ourdoc/vector3d.html#linear_combination

In addition to the 4-argument form detailed here, you may also call this method with 6 parameters in the form of:

Arguments:
weight1 - A weight or percentage.
vector1 - The first vector.
weight2 - A weight or percentage.
vector2 - The end point of the line.

What's the two extra arguments?

avariant

I'm pretty certain the material observer is completely wrong. The base class defines all but two methods incorrectly, and doesn't appear to use them all that correctly either.

scottlininger

Hello everyone,

Yes, we just launched some new docs for the Ruby API on the code site. These are the most accurate and up to date documentation we have ever had, and we hope that they are useful. (That being said, there is obviously still room for improvement.)

Your help in finding typos/missing details are very welcome! What isn't evident just from glancing at the site is that we have a new process internally for maintaining and updating the docs that will help us be more responsive to changes from here on out. (ThomThom, I've already pulled in your questions/corrections and will be pushing them out soon.)

By the way, HUGE thanks must be given to the people who worked on the wiki-enabled docs over the last couple of years. We integrated those community changes, and I'm currently doing a review to make sure that any changes from the last couple months are reflected in the new docs. Thank you! (The wiki docs are still up for now, and I will continue to monitor them for fixes.)

http://code.google.com/apis/sketchup/

Keep the corrections coming!

Chris Fullmer

Jim just pointed out this method and I noticed its missing:

Face.get_glued_instances

scottlininger

@chris fullmer said:

Jim just pointed out this method and I noticed its missing:

Face.get_glued_instances

Hey Chris (or Jim),

I don't think that's a standard method... (am I wrong?) I went to go add it to the docs, but I can't find it. Maybe it's part of some script out there?

Let me know,

remus

i thought the face.get_glued_instances method was added with the latest M1 release?

scottlininger

@chris fullmer said:

But is the SU Ruby code documented in a similar way to rDoc or something? Is it set up so that you guys can change the documentation, then run some rDoc like command on it and it automatically updates the website?

Yes.

We now generate the docs directly from source code comments, using something like rdoc. So we make a change to the source. It gets code reviewed and checked in. Then we push a button and it all goes onto the new site.

So... there are likely some "autogeneration" bugs that we will all discover over the coming weeks. Not to mention plain old human error. Which, by the way, is why the get_glued_instances method isn't up there. Remus is right, that was added with M1 and I am a bonehead. Fix coming!

Thx,

Chris Fullmer

Scott, I have question. This is coming from me, and I have little understanding of this documentation stuff. But is the SU Ruby code documented in a similar way to rDoc or something? Is it set up so that you guys can change the documentation, then run some rDoc like command on it and it automatically updates the website? Does that make sense?

Also, I like the tag that shows what version of SU each method is compatible with

Chris

scottlininger

@avariant said:

I'm pretty certain the material observer is completely wrong. The base class defines all but two methods incorrectly, and doesn't appear to use them all that correctly either.

Attached script demonstrates using the MaterialsObserver as documented. The base class may not have all of the methods correctly implemented, but SketchUp will call them.

As the docs say, the onMaterialRemoveAll does not fire in a particularly useful way.

Let me know if you find anything else! Thanks!

materialsObserverTest.rb

avariant

I guess my blanket comments about the MaterialObserver were not entirely accurate I was under the impression that unless some of the methods weren't explicitly implemented in the observer, SU would crash when it attempted to call the method since it's not implemented on the base. However, I seem to be wrong.
The documentation on onMaterialChange is a little inadequate. It says it is called whenever a material is altered, but it doesn't get called when you change size or color, just texture.

Jim

@unknownuser said:

@chris fullmer said:

Jim just pointed out this method and I noticed its missing:

Face.get_glued_instances

Hey Chris (or Jim),

I don't think that's a standard method... (am I wrong?) I went to go add it to the docs, but I can't find it. Maybe it's part of some script out there?

Let me know,

Scott, I got the information from the Release Notes page.

thomthom

Noticed in another thread that there seem to be some deviation related to send_action. http://www.sketchucation.com/forums/scf/viewtopic.php?f=180&t=16751&start=15#p135201

@unknownuser said:

"selectCameraOrbitTool:" doesnt work.
It's action name is simply OrbitTool.

Seems to be a couple more as well.

todd burch

@unknownuser said:

Hello everyone,

Yes, we just launched some new docs for the Ruby API on the code site. These are the most accurate and up to date documentation we have ever had, and we hope that they are useful. (That being said, there is obviously still room for improvement.)

Thanks Scott!

thomthom

Under the UI Classes section:
Tool, Toolbar and Webdialog all links to the Array class: http://code.google.com/apis/sketchup/docs/ourdoc/array.html

scottlininger

A bunch of fixes got pushed on Friday. Thanks again to folks who found typos, both from this thread and from the wiki documentation. Keep 'em coming!

I'm on vacation for the next week, so don't think I'm ignoring anyone if I don't reply for a few days. Thanks, guys!

thomthom

Thanks Scott.

I'm still puzzled about the six arguments for Vector3d.linear_combination.

vec = Geom;;Vector3d.linear_combination(x, xaxis, y, yaxis, z, zaxis)

How does this really work?

Chris Fullmer

Not to mention that the example at the bottom of the method does not actually include the method of linear combination. It only shows:

 vector1 = Geom;;Vector3d.new 1,0,0
 vector2 = Geom;;Vector3d.new 0,1,0

Chris Fullmer

PolygonMesh.add_polygon

The example shows how to use the add_point method.

thomthom

Wish we had red pens to scribble on the docs.

Jim

@thomthom said:

Wish we had red pens to scribble on the docs.

Great idea, really. I'm sure there are browser plugins that allow social sharing of sticky notes, or comments on web pages.