pyrevitlabs / pyRevit

Rapid Application Development (RAD) Environment for Autodesk Revit®
http://wiki.pyrevitlabs.io
GNU General Public License v3.0
1.3k stars 332 forks source link

Sum.pushputton\script.py #195

Closed eprunty closed 7 years ago

eprunty commented 7 years ago

Maybe here isn't the correct place for this but I edited line 80 to include square metres (for metric practices) as per below.

 print("TOTAL AREA OF ALL SELECTED ELEMENTS IS:\n{0} SQFT\n{1} ACRE\n{2} SQM".format(total, total / 43560, total/10.7639104))
eirannejad commented 7 years ago

Added your line to the script and pushed. Thanks for the feedback

eprunty commented 7 years ago

Thanks! This is by far the best Revit addin I've come across. Keep up the good work!

PS I have 4 scripts that may be of use to the pyRevit extension (add/remove selected revisions on selected sheets for 'by project' and 'by sheet' revisions). Can I send them to you for review/inclusion?

eirannejad commented 7 years ago

Of course brother. It might take me a couple of days to add them but I'm always happy to get new ideas and tools and make pyRevit better for everyone 😁

godoe commented 7 years ago

Hi Ehsan. I've just started looking into your tools. Amazing job man! Not sure it's the right place for this since the issue is closed, but about the Sum script, could you include meters for Length parameter since the default Value is in feet?

Thanks! Davide

eirannejad commented 7 years ago

Thanks a bunch pal. I actually started rewriting this script yesterday after your feedback and hopefully will finish today and publish soon. I’ll keep you posted on this.

Thanks for the kind words :)

On Jul 13, 2017, at 05:57, godoe notifications@github.com wrote:

Hi Ehsan. I've just started looking into your tools. Amazing job man! Not sure it's the right place for this since the issue is closed, but about the Sum script, could you include meters for Length parameter since the default Value is in feet?

Thanks! Davide

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/eirannejad/pyRevit/issues/195#issuecomment-315069813, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0XHN1TtMHLAMz-E4glIOowv9PZcMDlks5sNhQ1gaJpZM4NkJbD.

eirannejad commented 7 years ago

Resolved. Tool is improved now:

https://github.com/eirannejad/pyRevit/commit/c497d2dc358b5a0f751376ad9def05703a9338c2