[Plugin] bim-tools 0.13.4(june 22, 2015)

twillm

Thank you for the response.
I didn't make any changes to files.
Deleted plugin this morning and reinstalled through sketchucation plugin with similar error message.

steve69

Unpacked files in plugins folder..
bim-tools.rb
bim-tools folder

@ Sku 7.1 startup
Error Loading File C:/Program Files/Google/Google SketchUp 7/Plugins/bim-tools/bim-tools_loader.rb
No such file to load -- pathnameError Loading File bim-tools.rb
No such file to load -- pathnameError Loading File bim-tools_loader.rb
undefined method `strip' for nil:NilClass

Dan Rathbun

@steve69 said:

@ Sku 7.1 startup

Really? 6 year old version and 15 releases behind ? (Trimble has long stopped supporting v7.x.)

Does BIM-Tools even support v7.1 ?

Dan Rathbun

@Jan:
/bim-tools/lib/clsDefaultValues.rb:40

Single argument String subscripts are one of the breaking changes from Ruby 1.8 to 2.0.

The workaround is line[0,1], ie., [position, length] or give a range [0..0]

The single argument will definitely not work in SU2013 and earlier, as Ruby 1.8 returns an integer for the character's position in the ASCII character set.

brewsky

@steve69 said:

@ Sku 7.1 startup

Hmmm I have not been testing on old versions, I'll check it out, maybe it's easily fixed...

brewsky

@dan rathbun said:

Single argument String subscripts are one of the breaking changes from Ruby 1.8 to 2.0.

Hi Dan, thanks for helping out! That fixes one error!
SketchUp even seems to drops out here before the strip-string, on requiring the "pathname" module.
I guess that's one of the modules that was missing in the old 1.8 stripped ruby environment.

still some more warnings to wade through I see ...

Dan Rathbun

@brewsky said:

@dan rathbun said:

SketchUp even seems to drops out here before the strip-string, on requiring the "pathname" module.
I guess that's one of the modules that was missing in the old 1.8 stripped ruby environment.

Yes well it is a Standard Library class (derived from String, I believe.) And the Standard Ruby Library was not included until SketchUp 2014 (when it was released with Ruby 2.0.0.)

Before that, SketchUp was released with only the Core interpreter.

SketchUp 7.1 for Windows users can copy the Ruby 1.8.6-p287 DLL from v8 or v2013, back to the v7 program folder, or get it here at SketchUcation:
ruby-1.8.6-p287-i386-mswin32-dll.zip
(SketchUp 6 & 7 for Windows was released with the ancient Ruby v1.8.0 [initial release] of the Ruby interpreter.)

I have packaged the Ruby 1.8.6-p287 Standard Library up for Windows ONLY, as a normal RBZ SketchUp Extension archive. (It installs into a sub-folder as a plugin would.)
It only loads for SketchUp version 8..13 (or 7.x if the DLL is manually upgraded.)
https://github.com/DanRathbun/sketchup-ruby186-stdlib-extension/releases/tag/1

brewsky

I've been able to remove the dependency on pathname and it's running now on SU7 and SU8.

Although one (probably showstopper) problem remains. Auto-updating of modified objects does not work on SU7 because the "EntitiesObserver.onElementModified" method was added in SU8.
That means updating geometry must be done manually by disabling and re-enabling the observers (red/green button).

New version added to this post, I will add it to the plugin store when I also fixed the MAC issue, but that has to wait a little until I get my hands on a MAC

bim-tools-0.13.3.rbz

brewsky

Hi Dan,

I also removed a "tap"-call because it was added in ruby 1.9.
Would you consider this a good substitute? Or would it be much slower?

hash = {}.tap{ |r| bt_entities.each{ |ent| ent.properties_editable.each{ |k,v| (r[k]||=[]) << v } } }

hash = bt_entities.inject({}) {|h,ent| ent.properties_editable.each{ |k,v| (h[k]||=[]) << v}; h}

something I borrowed from:
http://stackoverflow.com/questions/5490952/merge-array-of-hashes-to-get-hash-of-arrays-of-values

Dan Rathbun

.
Well the 1st thing I had to do is proper indentation:

hash = {}.tap {|r|
  bt_entities.each {|ent|
    ent.properties_editable.each {|k,v|
      (r[k]||=[]) << v
    } 
  } 
}

and

hash = bt_entities.inject({}) {|h,ent|
  ent.properties_editable.each {|k,v|
    (h[k]||=[]) << v
  }
  h
}

Then the 2nd thing, is that Ruby is designed for easily read and understood code. (Why I like it so much.)

I do not understand why some coders go out of their way, to write minimalist hard to read and understand code. Come back 3 years from now and see if the coder remembers what the code is supposed to do, without spending a half hour consulting method documentation, etc.

3. Re., inject(), in the past in pre-YARV Ruby, it was notoriously slow. But things have gotten better.

4. My opinion of tap() is that it might be nifty, but it will serve mostly to contribute to ugly code.

5. I'd just do it simple like this:


hash = Hash;;new {|h,key| h[key]=[] }

bt_entities.each {|ent|
  ent.properties_editable.each {|k,v| hash[k] << v }
}

brewsky

@dan rathbun said:

.
I do not understand why some coders go out of their way, to write minimalist hard to read and understand code. Come back 3 years from now and see if the coder remembers what the code is supposed to do, without spending a half hour consulting method documentation, etc.

LOL Thank you very much! I totally agree with you. It indeed seemed a nifty piece of code I found on stackoverflow, but every time I needed to edit it, it took me half an hour to re-interpret it(and I'm still not 100% sure how the tap works).
Now you write it out it does not seem so difficult anymore
I should have added the indentation myself the moment I tried to understand what it did

Dan Rathbun

@brewsky said:

... and I'm still not 100% sure how the tap works

It is very similar to the map! / collect! methods for Enumerable objects. So you could use map! in place of tap for arrays.

But, in Ruby 2.0, tap is defined in Object, not just enumerable classes, so every object can "tap" into a block of code that can change and then return itself.

So tap is kind of like a general purpose "bang" method, but designed for especially for chaining so it always returns it's receiver object, whether it made changes or not. ("Bang" methods usually return nil when they do not make changes to the receiver, making them unsuitable for method chaining.)

brewsky

Fix for mac! Menu layout needs some work still.
Check out the plugin store

twillm

Thank you for Mac fix.
Successfully loaded and working.

brewsky

@twillm said:

Successfully loaded and working.

Great!

awadrummer

Does anyone know if this plugin is compatible with Sketchup Pro 2018? I'm looking to upgrade, but can't live without this one.

Thanks,

Chris B.

brewsky

@awadrummer said:

Does anyone know if this plugin is compatible with Sketchup Pro 2018?

I did a little test and I ran into some issues for su2018
I will try to find some time to look into them...

@awadrummer said:

I'm looking to upgrade, but can't live without this one.

Great to hear you like it!

awadrummer

@brewsky said:

@awadrummer said:

Does anyone know if this plugin is compatible with Sketchup Pro 2018?

I did a little test and I ran into some issues for su2018
I will try to find some time to look into them...

@awadrummer said:

I'm looking to upgrade, but can't live without this one.

Great to hear you like it!

Thank you Brewsky! The only issue I've found with 2018 is it freezes/takes a long time to process, but I generally have ALOT of lines. With that being said it is still usable. Thanks for your help and designing this awesome plugin!