State of Observers Redux — 18 August 2011

Krisidious · 12 Sept 2010, 13:55

they should have you on staff Thom...

Dan Rathbun · 12 Sept 2010, 14:01

MaterialsObserver.onMaterialRefChange

"When purging materials, or right-clicking a material in the Material Browser this event triggers one time for every entity with a material. This cause a long series of events to trigger unnessesary."

should read:
"When purging materials, or right-clicking a material in the Material Browser, this event triggers one time for every entity with a material. This causes a long series of events to trigger unnecessarily."

~

Jim · 12 Sept 2010, 14:08

Great work, and great resource. Thanks for doing it.

thomthom · 12 Sept 2010, 14:48

Thanks Dan, knew it was hiding somewhere.

Thanks for the spelling control - I didn't run a spell-checker or proofread yet.

@dan rathbun said:

"Depreciated" has no "i" in it. And the methods that were removed can not be "Deprecated", they are "Abandoned", "Deceased".. whatever they are no longer even in the Observer protoclass.

It's what the API docs said. I've not tested if the events are actually removed, or just aliased to their new names...

thomthom · 12 Sept 2010, 14:50

@dan rathbun said:

"Depreciated" has no "i" in it.

http://dictionary.reference.com/browse/Depreciated?r=75&src=ref&ch=dic

thomthom · 12 Sept 2010, 14:52

Oh, "Depreciated" refer to value...

grumble something about English and too similar words

thomthom · 12 Sept 2010, 15:04

Another issue, I imagine I read some reports on MaterialObserver.onMaterialSetCurrent not working as it should on OSX in some versions of SU?

Dan Rathbun · 12 Sept 2010, 15:06

@thomthom said:

It's what the API docs said. I've not tested if the events are actually removed, or just aliased to their new names...

I did test the MaterialObserver's Bulk change method. It has been removed from the superclass, so when you create an instance of a MaterialObserver subclass, it just is not there, and if you were to explicitly call the callback, a NoMethodError exception will be raised. Ie, there is no aliasing to the individual change callback(s).

So if you wanted to handle events in a bulk manner, you'd need to do it on the Ruby side, perhaps with a Ruby standard Observer class, or a bulkchange boolean flag variable and an Array of changed objects (each individual SU change callback would need to push it's object into the Array.)

Dan Rathbun · 12 Sept 2010, 15:12

@thomthom said:

Another issue, I imagine I read some reports on MaterialObserver.onMaterialSetCurrent not working as it should on OSX in some versions of SU?

YES.. I posted the answer that Scott gave me. He says it's logged as an internal bug.

See GG post: Materials.current on OS X
.

thomthom · 12 Sept 2010, 15:17

http://code.google.com/apis/sketchup/docs/ourdoc/pages.html#add_frame_change_observer
Sketchup::Pages.add_frame_change_observer(FrameChangeObserver.new)

This, surprisingly worked.

I'd thought you'd have to reference the model first...
Sketchup.active_model.pages.add_frame_change_observer(FrameChangeObserver.new)

thomthom · 12 Sept 2010, 15:19

@dan rathbun said:

@thomthom said:

Another issue, I imagine I read some reports on MaterialObserver.onMaterialSetCurrent not working as it should on OSX in some versions of SU?

YES.. I posted the answer that Scott gave me. He says it's logged as an internal bug.

See GG post: Materials.current on OS X
.

So it's bugged on SU6, SU7.0, SU 7.1?

thomthom · 12 Sept 2010, 15:20

@dan rathbun said:

@thomthom said:

It's what the API docs said. I've not tested if the events are actually removed, or just aliased to their new names...

I did test the MaterialObserver's Bulk change method. It has been removed from the superclass, so when you create an instance of a MaterialObserver subclass, it just is not there, and if you were to explicitly call the callback, a NoMethodError exception will be raised. Ie, there is no aliasing to the individual change callback(s).

So if you wanted to handle events in a bulk manner, you'd need to do it on the Ruby side, perhaps with a Ruby standard Observer class, or a bulkchange boolean flag variable and an Array of changed objects (each individual SU change callback would need to push it's object into the Array.)

bulk change event? Is that one that has been removed from the API?
That was a troublesome points when I made this chart - they recently removed some events from the API....

Dan Rathbun · 12 Sept 2010, 15:27

@thomthom said:

So it's bugged on SU6, SU7.0, SU 7.1?

Careful.. the collection getter method Materials.current IS bugged, the question remains, is the MaterialsObserver bugged as well. You'd need to test versions on the Mac to find out.
.

Dan Rathbun · 12 Sept 2010, 15:34

@thomthom said:

bulk change event? Is that one that has been removed from the API?
That was a troublesome points when I made this chart - they recently removed some events from the API....

Yes.. the MaterialsObserver.onMaterialRemoveAll callback was removed, as it never worked, and/or never got called. So there was no point in deprecating it.

thomthom · 12 Sept 2010, 15:42

@dan rathbun said:

@thomthom said:

bulk change event? Is that one that has been removed from the API?
That was a troublesome points when I made this chart - they recently removed some events from the API....

Yes.. the MaterialsObserver.onMaterialRemoveAll callback was removed, as it never worked, and/or never got called. So there was no point in deprecating it.

I'd like to add it for historic purpose - anyone remembers it parameters?

thomthom · 12 Sept 2010, 15:43

@thomthom said:

@dan rathbun said:

@thomthom said:

bulk change event? Is that one that has been removed from the API?
That was a troublesome points when I made this chart - they recently removed some events from the API....

Yes.. the MaterialsObserver.onMaterialRemoveAll callback was removed, as it never worked, and/or never got called. So there was no point in deprecating it.

I'd like to add it for historic purpose - anyone remembers it parameters?

Ah - already got it.
Thought there was an event named onBulkChange, like the SelectionObserver has.

thomthom · 12 Sept 2010, 15:45

aiaiai... sigh

thomthom · 18 Aug 2011, 21:57

Update to include SU 8.0 M1
http://www.thomthom.net/software/sketchup/observers/

Dan Rathbun · 18 Aug 2011, 22:19

You have that the AppObserver::onQuit is working...

It's is bugged in v8 (at least. And I did report a bug last beta cycle.)

It does not wait until the callbacks return, before shutting down the application.
Trying to use the callback to save plugin settings, and so forth, often fails for me.

Also, trying to dettach the observer itself from within onQuit causes a GPF (if I remember right.)
In addition coders should NOT open any "owned" windows (like WebDialogs and messageboxes,) during the callback, until they fix the 'non-blocking' issue.

Can't remember if v7 had any of these problems.

thomthom · 19 Aug 2011, 07:19

@dan rathbun said:

You have that the AppObserver::onQuit is working...

It's is bugged in v8 (at least. And I did report a bug last beta cycle.)

Ok - I'll add that. Never used it. All the observers I've not tried and not heard any reports on I assume is working.

@dan rathbun said:

InstanceObserver

@dan rathbun said:

what about Groups?

Groups and Images all are listed in model.definition.

@dan rathbun said:

(Note how I changed the example, from UI.messagebox to puts.

I wish the docs would do that. Messageboxes are double plus ungood - especially in observer events.