Turning a selection into a face?

tomot

@tig said:

@Tomot
Yes, please use your own Tomot namespace to avoid confusion with mt TIG one...

TIG is right, If you were working in an architectural office and did not know what a reveal was, your peers would wonder how you got the job.

tomot

@tig said:

@Tomot
Yes, please use your own Tomot namespace to avoid confusion with my TIG one...

TIG, I'm simply looking a your script and adding additional functonality, naturally and without question this is your script. This might turnout to be a good architectural addition and I would hope you could add it to your list of accomplished rubies

However as I already stated, while the Offset portion of the script has no problems, because the offset faces does not interfere with any adjoining face. The added "Reveal option" does have problems, because it introduces the problem of how "pushpull" can't effectively deal with push pulling shared faces. I recall there had been discussion about shared faces and pushpull but I could not find any solutions.

Dan Rathbun

@tomot said:

TIG is right, If you were working in an architectural office and did not know what a reveal was, your peers would wonder how you got the job.

Which, of course tells you correctly that I have not yet worked in architectural engineering. (Only mechanical, electrical, electronic and systems with a combination of those [ie, railway, vehicles, aircraft, and commercial broadcasting.])

Dan Rathbun

Referring to my previous code posting: HERE

To get multiple faces rather than the first face (which is not may not work, as the selection's order may not be returned in the order the user picked,) ...

      UI.add_context_menu_handler {|popup|
        unless Sketchup.active_model.selection.empty?
          sel = Sketchup.active_model.selection
          if sel.single_object? && sel[0].is_a?(Sketchup;;Face)
            face = sel[0]
            popup.add_item('Offset Face',10) { get_dist(face) }
          else
            faces = sel.grep(Sketchup;;Face)
            unless faces.empty? # (empty array if no face was selected)
              popup.add_item('Offset Selected Faces',11) {
                if get_values()
                  offset_multi_faces(faces)
                end
             }
            end
          end
        end
      }

The get_values() dialog method would need changing (from get_dist,) to just set the variables (as you did,) and return true to continue the command, or false if the user canceled the dialog, meaning they wished to cancel the command.

Then a new method offset_multi_faces(faces) would iterate the faces array, calling offset_a_face() for each face member in the array.

TIG

My Smart_offset method should return an array of the face[s] made by the offset [excluding the original face], which in your example will be just the inner new rectangular face.
So if you move the pushpull code into the block that is offsetting each face in turn to be something like:

faces.each{|face|
  newfaces=TIG;;Smart_offset.new(face, @dist, tidy)
  newfaces.each{|newface|newface.pushpull(@reveal)}if newfaces
}

it might work ???

tomot

@dan rathbun said:

selection's order may not be returned in the order the user picked,) ...

That is an entire new can of worms!....I don't know how SU implements a user select all picking order. I also see different results each time I use select all If you copy and paste each of the 4 attached cubes and run the script on each of the 4 cubes separate, sometimes the script gets the reveal right, most of the time it does not.

TIG

If you want to find the newly made reveal faces after the pushpull then here are the steps...
Before you pushpull make an array of all active faces:
facesIn=@model.active_entities.grep(Sketchup::Face)
Now do the pushpull and then find all new faces....
facesOut=@model.active_entities.grep(Sketchup::Face)-facesIn
Next remove all faces that share the same normal as the original ' face'
reveals=[]; facesOut.each{|f|reveals << f unless f.normal==face.normal}
Now the array ' reveals' contains all of the new reveal-faces that were made by that pushpull...
If there's a chance that the offsetting or pushpulling might delete the original ' face' then set norm=face.normal earlier in the code so that you have a record of it for use later on in ... f.normal==norm...

thomthom

@tig said:

reveals=[]; facesOut.each{|f|reveals << f unless f.normal==face.normal}

Can be written as one statement:
reveals = facesOut.select{ |f| f.normal != face.normal }

TIG

I know... but I thought it was a bit clearer what was happening the way I presented t

tomot

With thanks again to the 3 Ruby Amigos I will work on implementing your suggestions this weekend.