Support Forum

Hi,

in an interactor or script object it is a good Idea to select all elements only once, at the beginning (in the init). Here's an example for an interactor:

class Interactor(BaseInteractor):
    """Examples interactor"""

    def __init__(self,
                 coord_input             : AllplanIFWInput.CoordinateInput,
                 pyp_path                : str,
                 global_str_table_service: StringTableService,
                 build_ele_list          : list[BuildingElement],
                 build_ele_composite     : BuildingElementComposite,
                 control_props_list      : list[BuildingElementControlProperties],
                 modify_uuid_list        : ModificationElementList)                            :
        """Initialize the interactor"""

        self.coord_input               = coord_input
        self.doc                       = self.coord_input.GetInputViewDocument()
        self.all_elements              = AllplanBaseElements.ElementsSelectService.SelectAllElements(self.doc)

Best,
Bart

• I don't know, what is this something, that you were trying to do
  
• I don't know, what was inside the sel_elements list, when Allplan crashed
  

You have to provide more context.

Best,
Bart

Hi,

The code looks ok, but only in the case, there are solely 3D polylines in the document. There is no guarantee whatsoever, that the first element you are getting with __getitem__(0) will be a polyline.

Further more, actions like selecting all elements in the model should be a one-time action. create_elements gets executed over and over again in Allplan, basically every time you move the mouse around in the viewport. It's not a good idea to read all the data from the drawing file with every mouse move.

What you should do is:

• implement your logic in a function that gets executed only once, e.g. when a button is pressed. This is the case for on_control_event

  def on_control_event(build_ele: BuildingElement,
                       event_id: int,
                       doc: AllplanElementAdapter.DocumentAdapter) -> bool:
      if event_id == 1000:
          create_polyline_copies(doc, build_ele.Length.value)
  
      return False

• create your own function, e.g. create_polyline_copies(...)
  
• select all elements but then filter them. E.g. create a list by checking, whether each element geometry is really a Polyline3D, like:

  sel_elements = AllplanBaseElements.ElementsSelectService.SelectAllElements(doc)
  polyline_elements = [ele.GetModelGeometry() for ele in sel_elements if isinstance(ele.GetModelGeometry(), AllplanGeometry.Polyline3D)]
  

• Now you can do something with all these polylines. E.g. move them up:
  

  for polyline in polyline_elements:
      moved_polyline = AllplanGeometry.Move(polyline,vectormove)
  

  
  
• At the very end, you should create these polylines in the drawing file

  AllplanBaseElements.CreateElements(doc, AllplanGeometry.Matrix3D(), model_ele_list, [], None)

  
  

Have a look at the script. You can run it in Allplan. Create some 3D polylines beforehand and then run it and click "Go!".

Best,
Bart