sketchucation logo sketchucation
    • Login
    โ„น๏ธ Licensed Extensions | FredoBatch, ElevationProfile, FredoSketch, LayOps, MatSim and Pic2Shape will require license from Sept 1st More Info

    Hole plugin

    Scheduled Pinned Locked Moved Developers' Forum
    18 Posts 3 Posters 234 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.
    • N Offline
      njeremy2
      last edited by

      @thomthom said:

      I don't understand the rest of your code.

      mini = 10000
      What is this magic number?

      current_norm_reverse.length = 1000
      What is the purpose of this line?

      me too, that code is from rectangleHoletool.rb from the link in my first post

      1 Reply Last reply Reply Quote 0
      • N Offline
        njeremy2
        last edited by

        I made some test on that plugin.
        If the entity is longer than ~25000 cm (more than 25400cm, I just test it), you can make any hole. If it's smaller than that value, it working

        re-edit2 : I can make a hole in groups ! ๐Ÿ˜„ (many thanks thom !)
        Now I still have the first problem...

        1 Reply Last reply Reply Quote 0
        • thomthomT Offline
          thomthom
          last edited by

          The logic for finding the opposite face is somewhat odd. The magic number is no good design and puts on an artificial limit.
          I'm thinking that it'd be better to trace a ray to pick the opposite face. Shoot a ray from one of the vertices (or a calculated face centre) in the opposite direction of the face normal - see if it hit a face with a reversed normal. Then use that length to push-pull.

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

          1 Reply Last reply Reply Quote 0
          • thomthomT Offline
            thomthom
            last edited by

            Here is one alternative:


            No working correctly!!

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

            1 Reply Last reply Reply Quote 0
            • thomthomT Offline
              thomthom
              last edited by

              Meh! Doesn't work for faces outside the current context... ๐Ÿ˜ž

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

              1 Reply Last reply Reply Quote 0
              • thomthomT Offline
                thomthom
                last edited by

                This one works. ๐Ÿ˜„

                UPDATE: I'd forgotten to remove some old code. Please check the new version.


                Removed some redundant code!

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

                1 Reply Last reply Reply Quote 0
                • thomthomT Offline
                  thomthom
                  last edited by

                  I saw a tiger lurking...

                  http://images.bcdb.com/ad_im/wb/ltmm/putty_tat.jpg?u=

                  ๐Ÿ˜ฒ

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

                  1 Reply Last reply Reply Quote 0
                  • TIGT Offline
                    TIG Moderator
                    last edited by

                    I made an inept and incorrect comment, which I then decided to erase ๐Ÿ˜ฎ
                    You have fixed it now anyway...
                    BUT where'd it go ๐Ÿ˜•
                    Now it's back again... The art of illusion..........
                    "Now you see it, now you don't."

                    PS:
                    The latest code method's model = face.model is now redundant, because you iterate through the face.parent.entities to find the matching 'back-face'...

                    TIG

                    1 Reply Last reply Reply Quote 0
                    • thomthomT Offline
                      thomthom
                      last edited by

                      @tig said:

                      PS:
                      The latest code method's model = face.model is now redundant, because you iterate through the face.parent.entities to find the matching 'back-face'...

                      True that. (Won't bother to update for that one line though.)

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

                      1 Reply Last reply Reply Quote 0
                      • N Offline
                        njeremy2
                        last edited by

                        ahahah
                        AHAHAHAHAHA !!! Thank you thomthom !!

                        It works great !!!
                        Many many many .... (ctrl+C, ctrl+V) ... many thanks !! ๐Ÿ˜ ๐Ÿ‘

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

                        Advertisement