New API doc - typos and questions

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.

thomthom

Oh, yea... at least for Firefox. I mean to remember something like that.

thomthom

Could this be something? https://addons.mozilla.org/en-US/firefox/addon/2792
Anyone tried it? Or got better solutions?

Chris Fullmer

Point3d.project_to_plane:

http://code.google.com/apis/sketchup/docs/ourdoc/point3d.html#project_to_line

Shows the wrong example code (uses from project to line). and I know I gave the link for project_to_line. Its so you can see both methods simultaneously and see the error better. Thanks!

Chris

Chris Fullmer

So far I am really truly enjoying the new docs. They feel much better, thanks for the efforts to get them to where they are. I'm also excited that changes might be able to happen faster now too, that is great.

I'm noticing that sometimes in the methods list, when I click on a method, it takes me to the class index for that method. Other times it willtake me directly to the entry for that method within the class. I prefer the latter. But I don't know why sometimes it does one thing and sometimes it does another,

Chris

thomthom

http://code.google.com/intl/nb/apis/sketchup/docs/ourdoc/ui.html#inputbox

It doesn't mention how to allow drop-down options by using a third array with strings separating each option with a pipe | character.