[PR] [0.18-0.21_pre] Arch Stair - More Geometry Control? Profile Generator

A forum dedicated to the Draft, Arch and BIM workbenches development.
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: Arch Stair - Control Length of Landing? More Geometry Control? + Bug Report ['pre-Pull Request' ?]

Post by paullee »

To the contrary, Yorik, Kunda etc. is spending their extra time just to help me to commit the few changes I manage to made :)

I always think Yorik can do the changes I made with 5 to 6 weeks, if useful at all, in a few days, or just in a few hours.

Anyway, I am having 2nd attempt to merge the HalfTurnLeft 'basic feature', hope it is found useful.

Join the camp, it's fun, when you have a gap having a rest not drafting designing.
User avatar
yorik
Founder
Posts: 13660
Joined: Tue Feb 17, 2009 9:16 pm
Location: Brussels
Contact:

Re: Arch Stair - Control Length of Landing? More Geometry Control? + Bug Report ['pre-Pull Request' ?]

Post by yorik »

No problem! I'm happy to see someone wanting to spend some sweat on those stairs... They are a very tricky geometry problem
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [Merging Pull-request] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

Thanks to all helps and comments here.

Yorik is so kind to accept a nos. of PR / 10 commits for the 0.18_pre :)

As I start playing the ArchStairs.py based on 0.17 Git13522, I have spent more than 1 week just to 'port' part of it to 0.18_pre the current master (Python newbie) :oops:
(another 1 week to read github / git related tutorials!)

ArchStairs.py 'Revision' in 0.18_pre
  • PR/Commits Accepted :
    Up to around 10.7.2018 discussion in the thread
  • PR/Commits Proposed :
    Up to around 16.7.2018 discussion in the thread
  • Commits in Pipeline :
    Up to around 18.7.2018 discussion in the thread
  • Commits yet to be 'ported' to 0.18_pre :
    About 4 more; Up to around 1.8.2018 discussion in the thread
Interim Status

Now it seem there is no point of return for me as I think many of the 'features' are not fully done, not even for the 'latest' development I did (as of 1.8.2018) :lol:

So it would be buggy at the moment, and features half-done still in near future. Hope peoples still find it useful and comments welcome to improve it (codes from more peoples?)

Cheers.
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [Pull-request Merged upto 1.8] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

Now:

ArchStairs.py 'Revision' in 0.18_pre Cheers.
User avatar
zbigg
Posts: 136
Joined: Tue Dec 19, 2017 1:11 pm

Re: [PR Merged upto 1.8.2018] Arch Stair - More Geometry Control? Bug Report

Post by zbigg »

Hello,
May I add my two cents to the discussion?
I would suggest some feature, cant implement it personally so far-just learning JS and Python-but as soon as I reach reasonable skills
I promise to help extend FC, starting with ArchModule first.
The thing is: maybe its only country specific feature but I suppose it may be very useful.
Aside of local regulations reg.stairs there is a 'rule of thumb' for it: 2h+s = 60-65
Meaning: h-height of a step, s-width/depth of a step and above formula '2h+s' should be fulfilled in range of 60-65.
Suppose dims are in centimeters. Above guarantee most usable and comfortable stairs in given conditions.
cheers,
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [PR Merged upto 1.8.2018] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

zbigg wrote: Thu Aug 30, 2018 6:29 am Hello,
May I add my two cents to the discussion?
I would suggest some feature, cant implement it personally so far-just learning JS and Python-but as soon as I reach reasonable skills
I promise to help extend FC, starting with ArchModule first.
The thing is: maybe its only country specific feature but I suppose it may be very useful.
Aside of local regulations reg.stairs there is a 'rule of thumb' for it: 2h+s = 60-65
Meaning: h-height of a step, s-width/depth of a step and above formula '2h+s' should be fulfilled in range of 60-65.
Suppose dims are in centimeters. Above guarantee most usable and comfortable stairs in given conditions.
Thanks for comment! and participation!

I am also python newbie, 1st attempt to write something for Arch!
(With help from York and all other people here)

I think York / FC already had a "BlondelRatio" - "The Blondel ratio indicates comfortable stairs and should be between 62 and 64cm or 24.5 and 25.5in")) - Check the property box.

Code: Select all

    obj.BlondelRatio = obj.RiserHeight.Value*2+obj.TreadDepth.Value  
Have lot of things that my elementary python knowledge does not let me to implement - e.g. simple thing like QT dialog box.

Do help!
User avatar
zbigg
Posts: 136
Joined: Tue Dec 19, 2017 1:11 pm

Re: [PR Merged upto 1.8.2018] Arch Stair - More Geometry Control? Bug Report

Post by zbigg »

Great! That should do the job. /Although its an option anyway )) /
cheers,
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [PR] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

Hi, implemented different width for the (mult-edge) landing (as no much idea how other features should be done :) )

A new Property 'Width of Landing' is added

It is a list of width which set the width of different 'edge' of a Landing
- this is for 2nd edge and after, 1st edge follow original Width property
- 'default' setting in original Width property would be used if a '0' (zero) setting is made for any 'edge'


Below ArchStairs.py is for 0.18_pre
Version for 0.17 is available if anyone is intertested)

It is implemented with App::PropertyFloatList ... as App::PropertyLengthList is not yet available

See if there is any comments before a PR is being made.

Thanks.
Screenshot from 2018-09-23 12-08-22.png
Screenshot from 2018-09-23 12-08-22.png (289.35 KiB) Viewed 1458 times
Screenshot from 2018-09-23 11-49-35.png
Screenshot from 2018-09-23 11-49-35.png (247.08 KiB) Viewed 1458 times
Screenshot from 2018-09-23 12-13-28.png
Screenshot from 2018-09-23 12-13-28.png (334 KiB) Viewed 1458 times
Attachments
ArchStairs.py
(51.62 KiB) Downloaded 54 times
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [PR] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

Some more images, landing & flight in different width:-
Screenshot from 2018-09-25 09-40-14.png
Screenshot from 2018-09-25 09-40-14.png (172.77 KiB) Viewed 1434 times
Screenshot from 2018-09-25 09-41-19.png
Screenshot from 2018-09-25 09-41-19.png (227.06 KiB) Viewed 1434 times
Screenshot from 2018-09-25 09-42-45.png
Screenshot from 2018-09-25 09-42-45.png (252.08 KiB) Viewed 1434 times
Screenshot from 2018-09-25 09-36-26.png
Screenshot from 2018-09-25 09-36-26.png (234.38 KiB) Viewed 1434 times
paullee
Veteran
Posts: 5118
Joined: Wed May 04, 2016 3:58 pm

Re: [PR] Arch Stair - More Geometry Control? Bug Report

Post by paullee »

ArchPipe objects named as "Rail"
Screenshot from 2018-09-28 06-30-19.png
Screenshot from 2018-09-28 06-30-19.png (284.49 KiB) Viewed 1412 times
Post Reply