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

---

[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