• Login
sketchucation logo sketchucation
  • Login
๐Ÿค‘ SketchPlus 1.3 | 44 Tools for $15 until June 20th Buy Now

Return values

Scheduled Pinned Locked Moved Developers' Forum
14 Posts 3 Posters 871 Views 3 Watching
Loading More Posts
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • T Offline
    thomthom
    last edited by 24 Jul 2012, 21:23

    @dan rathbun said:

    You must convert back mm, using .to_mm, but because you cubed it, you have to "cube" the conversion

    It's better to use .to_l - then when that Length instance is outputted as string it's printed in the user's units.
    As for area - use Sketchup.format_area

    Unfortunately there is no method for volume - they left that one out. So in that particular instance you need to write the conversion from cubed inches to user units yourself.

    Thomas Thomassen โ€” SketchUp Monkey & Coding addict
    List of my plugins and link to the CookieWare fund

    1 Reply Last reply Reply Quote 0
    • T Offline
      thomthom
      last edited by 24 Jul 2012, 21:28

      prompts = ["Width mm", "Breadth mm", "Height mm"] defaults = ["100", "100", "100"]

      Let SketchUp handle the units for you. If you feed the input box Length objects as defaults - it converts the user input to Length's for you.
      defaults = [100.mm, 100.mm, 100.mm]

      It's better to work with units in SketchUp's units - internally inches - and always ensure you have Length objects when you output to the user. That way the user gets the units in his/her preferred unit and you don't have to do a thing.

      Thomas Thomassen โ€” SketchUp Monkey & Coding addict
      List of my plugins and link to the CookieWare fund

      1 Reply Last reply Reply Quote 0
      • D Offline
        Dan Rathbun
        last edited by 25 Jul 2012, 04:14

        There was another topic on this issue, I think I may have posted a method in there.
        Or.. hmmm... It may have a "[Code]" prefix in the title.
        ADD: found it: Volume of Multiple Groups
        You might get some ideas from that, and there is a link to one of TIG's old utilities.


        Anyway.. TT... One thing you or TIG have not done yet is a "Units: The Lost Manual" ๐Ÿ˜†

        I'm not here much anymore.

        1 Reply Last reply Reply Quote 0
        • T Offline
          thomthom
          last edited by 25 Jul 2012, 07:49

          @dan rathbun said:

          Anyway.. TT... One thing you or TIG have not done yet is a "Units: The Lost Manual" ๐Ÿ˜†

          That is in fact on my list to write up.

          Thomas Thomassen โ€” SketchUp Monkey & Coding addict
          List of my plugins and link to the CookieWare fund

          1 Reply Last reply Reply Quote 0
          • D Offline
            Dan Rathbun
            last edited by 25 Jul 2012, 08:39

            @paulg said:

            Thanks Dan, although .to_mm.to_mm.to_mm looks like some kind of comedy code. I gotta say Ruby seems the most obscure, non-sensical language I have used so far.

            Of course it does. It's not Ruby's fault, that is not Ruby best practice.

            There are many "missing" methods in various API classes.

            In fact what we are missing here, are Area and Volume classes. When you multiply 2 Length instance objects, they get converted to Float.. and then you cannot tell what the math ops were to end up with the value.

            In other words... math on a Length object, by any other Numeric subclass, should return a new Length object. (Like adding the frontage of building lots along a right-of-way, or math ops with wall lengths to arrive at a perimeter length.)

            Multiplying two Length objects, should return a new Area object.

            Multiplying 3 Length objects, OR an Area object by a Length object, should return a new Volume object.

            However, multiplying Area objects, by other non-length Numeric objects, should result in a new larger Area object.

            .. etc.

            Problem is, current version of SketchUp would not know how to handle these new classes.

            I'm not here much anymore.

            1 Reply Last reply Reply Quote 0
            • topic:timeago-later,23 days
            • T Offline
              thomthom
              last edited by 17 Aug 2012, 11:06

              @dan rathbun said:

              Anyway.. TT... One thing you or TIG have not done yet is a "Units: The Lost Manual" ๐Ÿ˜†

              Link Preview Image
              Dealing with Units in SketchUp

              There are extensions to the base classes in SketchUpโ€™s Ruby API which often new SketchUp plugin developers overlook. If you arenโ€™t aware of them you might find yourself reinventing manyโ€ฆ

              favicon

              Procrastinators Revolt! (www.thomthom.net)

              Thomas Thomassen โ€” SketchUp Monkey & Coding addict
              List of my plugins and link to the CookieWare fund

              1 Reply Last reply Reply Quote 0
              • D Offline
                Dan Rathbun
                last edited by 17 Aug 2012, 11:56

                ๐Ÿ‘ ๐Ÿ‘

                NO faults that I can see, after a quick read.

                I'm not here much anymore.

                1 Reply Last reply Reply Quote 0
                • T Offline
                  thomthom
                  last edited by 17 Aug 2012, 12:02

                  @dan rathbun said:

                  :thumb: ๐Ÿ‘

                  NO faults that I can see, after a quick read.

                  I'm sure there's some input to be made on the library I posted at GitHub - linked at the bottom. I'd collected it from various helper methods from different plugins of mine. The volume and area functions was originally not in separate classes - but I liked idea idea of Area and Volume as separate classes.

                  Thomas Thomassen โ€” SketchUp Monkey & Coding addict
                  List of my plugins and link to the CookieWare fund

                  1 Reply Last reply Reply Quote 0
                  • D Offline
                    Dan Rathbun
                    last edited by 17 Aug 2012, 12:36

                    @thomthom said:

                    @dan rathbun said:

                    ... but I liked idea idea of Area and Volume as separate classes.

                    I also ! (as I said above.)

                    And the API needs these, and the app needs to use them.

                    I'm not here much anymore.

                    1 Reply Last reply Reply Quote 0
                    • T Offline
                      thomthom
                      last edited by 17 Aug 2012, 12:43

                      And locale handling for Floats.

                      Thomas Thomassen โ€” SketchUp Monkey & Coding addict
                      List of my plugins and link to the CookieWare fund

                      1 Reply Last reply Reply Quote 0
                      • T Offline
                        thomthom
                        last edited by 17 Aug 2012, 12:45

                        @dan rathbun said:

                        And the API needs these, and the app needs to use them.

                        Yes - agree. Your idea about multiply two Length to get and Area, and Area with Length to get a Volume is very elegant. I'd very much like to see this into SU9.

                        Thomas Thomassen โ€” SketchUp Monkey & Coding addict
                        List of my plugins and link to the CookieWare fund

                        1 Reply Last reply Reply Quote 0
                        • 1 / 1
                        1 / 1
                        • First post
                          13/14
                          Last post
                        Buy SketchPlus
                        Buy SUbD
                        Buy WrapR
                        Buy eBook
                        Buy Modelur
                        Buy Vertex Tools
                        Buy SketchCuisine
                        Buy FormFonts

                        Advertisement