[Plugin] Roof.rb

tylovset

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

TIG

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

sherwood111

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 !

utiler

Thanks for doing this, TIG. very much appreciated; as always.....

pbacot

Thanks TIG!!!!

seasdes

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?

facer

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

TIG

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

Dave R

It works as expected for me. Thanks TIG!

seasdes

Thanks so much TIG.
I assume that the help file for V3 is still current

TIG

The change was to make it compatible with v2017.
Its usage is unchanged...

tspco

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.

TIG

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...

tspco

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.

tylovset

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

TIG

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...

mariocha

Anyone else having this bug with Gable Roof when using ^Height ?

Error: #<NoMethodError: undefined methodlength' for nil:NilClass>
C:/Users/---------/AppData/Roaming/SketchUp/SketchUp 2018/SketchUp/Plugins/Roof.rb:1725:in dialog_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:in onLButtonDown'

^Height.png

TIG

~~Have you tried ^55" ?~~
EDIT: Forget that.
It's nothing to do with it per se, although it only manifests itself using ^Height...
There's a weird typo in the gable making code which I need to resolve...
Watch this space...

TIG

Here's v5.0 https://sketchucation.com/pluginstore?pln=Roof It fixes a typo in the Gable by ^Height code which caused a failure.
this was a long standing typo, which has taken several years to find and be reported on !

The code is also re-signed for v2018 compatibility...

Ahmed0007

Please Update Follow me & Rotate