New API doc - typos and questions

[thomthom]

I'm looking at the new API docs and I noted an inconsistency in the documentation. I thought we might collect them here as they're discovered and hopefully we'll see them addressed.

The example for Set.include? http://code.google.com/apis/sketchup/do ... ml#include? uses a different method name:

Code: Select all

status = set.contains? object

[thomthom]

Vector3d.linear_combination
http://code.google.com/apis/sketchup/do ... ombination

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?

[Chris Fullmer]

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

Code: Select all

Face.get_glued_instances

[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!

[ScottLininger]

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

Code: Select all

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,

[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

[remus]

[ruby newb]i thought the face.get_glued_instances method was added with the latest M1 release?[/ruby newb]

[ScottLininger]

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,

[ScottLininger]

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!

[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]

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

Code: Select all

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. viewtopic.php?f=180&t=16751&start=15#p135201

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

Seems to be a couple more as well.

[Todd Burch]

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/do ... array.html