[Plugin] Roof.rb
-
Hello,
Is anyone having problems using "roof" in 2017 pro ?
I get this in the ruby console (for a hipped roof) :
[it works fine in 2016 pro]Error: #<TypeError: reference to deleted Face>
C:/Users/apache3/AppData/Roaming/SketchUp/SketchUp 2017/SketchUp/Plugins/Roof.rb:287:inouter_loop' C:/Users/apache3/AppData/Roaming/SketchUp/SketchUp 2017/SketchUp/Plugins/Roof.rb:287:in
hip'
C:/Users/apache3/AppData/Roaming/SketchUp/SketchUp 2017/SketchUp/Plugins/Roof.v4.rb:4088:inblock in <top (required)>' SketchUp:1:in
call' -
It currently does not work in v2017 because of the way they changed how SketchUp references/IDs exploded entities.
It was reported in a separate post just before Xmas, it's on my list of things to fix...
Meanwhile you can still use it in v2016 and import the roof geometry into a v2017 model -
TIG, Thanks for your reply and all that You do.
Yep, I'm finding that I have to utilize a
similar workflow as a workaround with other
plugins too ... (in my transition to 2017).[slightly off subject]
In general, I also get the annoying extension
manager popup every session :
"There are 'x' extensions ready to be updated"
However, they won't update for various reasons.Onward and upward, as they say.
-
Hi TIG, sherwood111
To make it work in su2017, simply comment out the line 'gp.explode' two places in the file.
The code is referring to entities from a group after an explode, which is not allowed...Alternatively, move the lines mentioned down, e.g. before 'if sof==0###v3.0', to get rid of the group when no longer referred to. There are also other groups left in the model created, not sure if that is intentional.
Just a small contribution to thank for a fantastic plugin, has been most valuable!
-tyge
-
Here's v4.0
There are some minor tweaks and it has been adjusted for full certification in v2016/2017 and also full compatibility with v2017's id changes etc.
http://sketchucation.com/pluginstore?pln=Roof -
Excellent ..
Yes, it is a fantastic and valuable plugin (installed and working in 2017).
tylovset, Thank You for your time and input . . .
TIG, Thank You for the plugin, your time and update.
Very Much Appreciated ! -
Thanks for doing this, TIG. very much appreciated; as always.....
-
Thanks TIG!!!!
-
Hello TIG
I have installed V4.0 into SU2017.
I drew a simple rectanglular face and tried to create a hipped roof. However, I get the attached error message. Any ideas as to what I am doing wrong?
-
SketchUp 2016 W10 32 bit
I get the same error message in SketchUp 2016 Roof v4.0
Tried also for version Roof v3.7 and got the same error message
-
This limitation is set out in the Help...
Your selected flat face cannot be at z=0 [i.e. on the ground].
Also it's best if the seed-face is looking upwards !
Yours is facing down AND at z=0
Try again, avoiding those limitations, i.e move the face up off the ground and reverse it so it's upwards... and it should then work [unless of course your face has such a complicated outline that it is just not possible to add a single-pitch hipped roof over it... but that's another issue]...
-
It works as expected for me. Thanks TIG!
-
Thanks so much TIG.
I assume that the help file for V3 is still current -
The change was to make it compatible with v2017.
Its usage is unchanged... -
Tonight I saw an update for Tig's Roof plugin, I installed it, and it worked a couple of times. then I get this, Maybe I did something wrong, I don't know for sure.
-
This is a known limitation.
Please read the recent posts...
The seed-face used for a hipped roof must NOT be at z=0 [when do you see a roof on the ground ?], and it should also be oriented to face upwards.
Your test examples seem to involve a rectangle drawn directly onto the ground and it is facing downwards too !
Extrude that face upwards into a 'box', select its top face, and now a hipped roof should form OK... -
That makes sense, when I mentioned it worked, I was in another project and drew the roof on the floor (above 0Z) of that model, just experimenting with the update.
-
TIG,
I made a shape where I can create a Mansard roof, but it says it cannot make a Hipped roof.
I mention this because the algorithms seems similar. In fact, I can set "Height of Lower Roof" to only 1cm and it looks like a regular Hipped Roof. However, setting the height to 0 fails like Hipped roof.Could it be a bug? Here's the file: https://1drv.ms/u/s!AnwI6G-MJcza1X_aCfVxdSqVB69t
-
The code for both roof-types is very similar, so I can't see why the hipped one fails.
However, it was easily fixed: I deleted the top face and drew over a single edge, so that SketchUp added back the face - as explained in the error-message sometimes a face's edge-order/loop-direction becomes confused in the SKP's data-base.
That replacement face then made a hipped roof without any difficulty...
-
Anyone else having this bug with Gable Roof when using ^Height ?
Error: #<NoMethodError: undefined method
length' for nil:NilClass>
C:/Users/---------/AppData/Roaming/SketchUp/SketchUp 2018/SketchUp/Plugins/Roof.rb:1725:indialog_gable' C:/Users---------/AppData/Roaming/SketchUp/SketchUp 2018/SketchUp/Plugins/Roof.rb:2376:in
gable'
C:/Users/--------/AppData/Roaming/SketchUp/SketchUp 2018/SketchUp/Plugins/Roof.rb:3919:inonLButtonDown'
Advertisement