Fixed some long-lasting update issue with the GUI

- Also improved the recently added Mohr-Coulomb theory feature
- Fixed older bug with Estimate Cluster Radius

Author: KaiKostack

kk_bullet_constraints_builder.zip

@@ -196,6 +196,8 @@ def getConfigDataFromScene(scene):
 
     ### Preprocessing Tools
 
+    props.update_lock = 1  # Suppress automatic updating to set a new property value
+    
     if "bcb_prop_preprocTools_aut" in scene.keys():
         props.preprocTools_aut = scene["bcb_prop_preprocTools_aut"]
 
@@ -406,6 +408,11 @@ def getConfigDataFromScene(scene):
         props.detonPullBackDelay = scene["bcb_prop_detonPullBackDelay"]
     if "bcb_prop_detonGroundReflect" in scene.keys():
         props.detonGroundReflect = scene["bcb_prop_detonGroundReflect"]
+
+    props.update_lock = 0
+    
+    ###### Update global vars from menu properties
+    props.props_update_globals()
 
     #if len(warning): return warning
 

kk_bullet_constraints_builder/build_data.py

@@ -55,7 +55,9 @@ def initGeneralRigidBodyWorldSettings(scene):
             scene.render.fps_base = 1
             print("New Frame Rate set to:", scene.render.fps)
         else:
+            props.update_lock = 1  # Suppress automatic updating to set a new property value
             props.stepsPerSecond = scene.render.fps *int(props.stepsPerSecond /scene.render.fps)
+            props.update_lock = 0
             print("New Solver Steps set to:", props.stepsPerSecond)
             if props.stepsPerSecond %scene.render.fps != 0 or scene.render.fps_base != 1:
                 fps = props.stepsPerSecond /int(props.stepsPerSecond /scene.render.fps)

kk_bullet_constraints_builder/builder_prep.py

@@ -189,6 +189,7 @@ def importConfigData(scene):
         return 1
     else:
         props = bpy.context.window_manager.bcb
+        props.update_lock = 1  # Suppress automatic updating to set a new property value
         i += 1
         props.stepsPerSecond = configData[i]; i += 1
         props.solverIterations = configData[i]; i += 1
@@ -227,7 +228,12 @@ def importConfigData(scene):
         props.detonBlastWaveVel = configData[i]; i += 1
         props.detonPullBackDelay = configData[i]; i += 1
         props.detonGroundReflect = configData[i]; i += 1
+        props.update_lock = 0
+        ###### Update global vars from menu properties
+        props.props_update_globals()
+
         global_vars.elemGrps = configData[i]; i += 1
+        
         return 0
 
 ################################################################################   

kk_bullet_constraints_builder/file_io.py

@@ -40,7 +40,7 @@
 
 def dpifac100():
     prefs = bpy.context.user_preferences.system
-    if hasattr(prefs, 'pixel_size'):  # python access to this was only added recently, assume non-retina display is used if using older blender
+    if hasattr(prefs, 'pixel_size'):  # Python access to this was only added recently, assume non-retina display is used if using older blender
         retinafac = bpy.context.user_preferences.system.pixel_size
     else:
         retinafac = 1
@@ -50,31 +50,34 @@ class bcb_report(bpy.types.Operator):
     bl_idname = "bcb.report"
     bl_label = "Info"
     bl_description = "Report message operator"
+
+    message = bpy.props.StringProperty()
+    icon = bpy.props.StringProperty()
+    
     def execute(self, context):
         return {'FINISHED'}
+    
     def invoke(self, context, event):
         wm = context.window_manager
-        props = context.window_manager.bcb
-        print(props.message)
+        print(self.message)
         # Calculate safe width in pixel from char count of the string
-        strSize = len(props.message)
+        strSize = len(self.message)
         # widthPx = (strSize*12+40) /dpifac100()  # Approx. width for capitals
         widthPx = (strSize*8+30) /dpifac100()   # Approx. width for normal text
         #return wm.invoke_props_dialog(self)
         return wm.invoke_popup(self, width=widthPx, height=300)
+    
     def draw(self, context):
-        layout = self.layout
-        props = context.window_manager.bcb
-        message = props.message
-        row = layout.row()
-        row.label(text=message, icon="ERROR")
+        row = self.layout.row()
+        row.label(text=self.message, icon=self.icon)
+        row.alert = True  # Red background
 
 ########################################
 
 class bcb_add_preset(bpy.types.Menu):
     bl_idname = "bcb.add_preset"
     bl_label = "Available Presets"
-    bl_description = "Duplicates selected element group"
+    bl_description = "Adds a preset element group to list"
     def draw(self, context):
         layout = self.layout
         #layout.operator("wm.open_mainfile")
@@ -148,9 +151,6 @@ def draw(self, context):
             split2 = split.split(align=1)
             split2.operator("bcb.set_config", icon="NEW")
 
-        ### Update global vars from menu related properties
-        props.props_update_globals()
-
 ########################################
 
 class bcb_panel_preprocessing_tools(bpy.types.Panel):

kk_bullet_constraints_builder/formula_props.py

@@ -45,7 +45,7 @@
 from tools import *            # Contains smaller independently working tools
 
 ################################################################################   
-         
+
 class OBJECT_OT_bcb_set_config(bpy.types.Operator):
     bl_idname = "bcb.set_config"
     bl_label = ""
@@ -203,7 +203,8 @@ class OBJECT_OT_bcb_export_ascii_fm(bpy.types.Operator):
     use_handler = int_(default = 0)
     def execute(self, context):
         if not hasattr(bpy.types.DATA_PT_modifiers, 'FRACTURE'):
-            self.report({'ERROR'}, "Fracture Modifier not available in this Blender version. Visit kaikostack.com/fracture for the FM-enabled Blender version.")  # Create popup message
+            # Create popup message
+            self.report({'ERROR'}, "This feature requires a Blender version with Fracture Modifier. Visit kaikostack.com/fracture for the FM-enabled Blender version.")
         else:
             ###### Execute main building process from scratch
             scene = bpy.context.scene
@@ -337,8 +338,11 @@ def execute(self, context):
             if self.menuIdx < 0:
                 # Call menu
                 bpy.ops.wm.call_menu(name="bcb.add_preset")
-            else:
+            print("DEBUG button", self.menuIdx)
+            if self.menuIdx >= 0:
                 props = context.window_manager.bcb
+                # Update global vars (only for initializing on new scene)
+                props.props_update_globals()
                 elemGrps = global_vars.elemGrps
                 # Add element group (syncing element group indices happens on execution)
                 elemGrps.append(presets[self.menuIdx].copy())

kk_bullet_constraints_builder/global_props.py

@@ -558,24 +558,35 @@ def monitor_checkForChange(scene):
 
                     ### Dynamic change of breaking thresholds depending on pressure (Mohr-Coulomb theory)
                     if qMohrCoulomb:
-                        # Get force acting on the compressive constraints in connection
-                        force = abs(consts[0].rigid_body_constraint.appliedImpulse()) *rbw_steps_per_second /rbw_time_scale  # Conversion from impulses to forces
+                        # Find the maximum force of the compressive constraints in connection
+                        forceMax = 0
+                        for const in consts:
+                            con = const.rigid_body_constraint
+                            ### For Point and Fixed costraints
+                            ### For Generic constraints
+                            # Compressive constraints
+                            if con.type != 'GENERIC' \
+                            or con.use_limit_lin_x:
+                                force = abs(con.appliedImpulse()) *rbw_steps_per_second /rbw_time_scale  # Conversion from impulses to forces
+                                if force > forceMax: forceMax = force
                         # Compute new breaking threshold incease based on force
                         # σ = F /A
                         # τ = c +σ *tan(ϕ)
-                        brkThresInc = force /contactArea *0.577 *mul
+                        brkThresInc = forceMax /contactArea *0.577 *mul
                         # Modify constraints
                         for i in range(1, len(consts)):  # We know that first constraint is always pressure
                             con = consts[i].rigid_body_constraint
-                            # Set override to shear connections in connection
-                            if con.type != 'GENERIC':  # For Point and Fixed costraints
+                            ### For Point and Fixed costraints
+                            ### For Generic constraints
+                            # Tensile constraints - Comment this line out to include all constraints
+                            # Shear constraints - Comment this line out to include all constraints
+                            # Bend constraints - Comment this line out to include all constraints
+                            if con.type != 'GENERIC' \
+                            or con.use_limit_lin_x \
+                            or con.use_limit_lin_y or con.use_limit_lin_z \
+                            or con.use_limit_ang_y or con.use_limit_ang_z:
+                                # Apply breaking threshold incease
                                 con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
-                            else:  # For Generic constraints
-                                if con.use_limit_lin_y or con.use_limit_lin_z:  # Shear constraints - Comment this line out to include all constraints
-                                    con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
-                                # Set override to bend connections in connection
-                                if con.use_limit_ang_y or con.use_limit_ang_z:  # Bend constraints - Comment this line out to include all constraints
-                                    con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
 
 #                    ### Modify limits from applied forces
 #                    strainDist = .001 # Maximum linear strain for the given breaking threshold
@@ -634,7 +645,7 @@ def monitor_checkForChange(scene):
 #                    strain = abs(force) /brkThres  # Normalized to breaking threshold
 #                    if strain > strainMax: strainMax = strain
 #            strainIters = iterMin +(iterMax -iterMin) *strainMax  # Compute iterations from strain
-#            # Set override only to shear connections in connection
+#            # Set override only to shear constraints in connection
 #            for const in consts:
 #                con = const.rigid_body_constraint
 #                if con.use_limit_lin_y or con.use_limit_lin_z:  # Shear constraints - Comment this line out to include all constraints
@@ -847,25 +858,35 @@ def monitor_checkForChange_fm(scene):
 
                 ### Dynamic change of breaking thresholds depending on pressure (Mohr-Coulomb theory)
                 if qMohrCoulomb:
-                    # Get force acting on the compressive constraints in connection
-                    force = abs(consts[0].appliedImpulse()) *rbw_steps_per_second /rbw_time_scale  # Conversion from impulses to forces
+                    # Find the maximum force of the compressive constraints in connection
+                    forceMax = 0
+                    for con in consts:
+                        ### For Point and Fixed costraints
+                        ### For Generic constraints
+                        # Compressive constraints
+                        if con.type != 'GENERIC' \
+                        or con.use_limit_lin_x:
+                            force = abs(con.appliedImpulse()) *rbw_steps_per_second /rbw_time_scale  # Conversion from impulses to forces
+                            if force > forceMax: forceMax = force
                     # Compute new breaking threshold incease based on force
                     # σ = F /A
                     # τ = c +σ *tan(ϕ)
-                    brkThresInc = force /contactArea *0.577 *mul
+                    brkThresInc = forceMax /contactArea *0.577 *mul
                     # Modify constraints
                     for i in range(1, len(consts)):  # First constraint is always pressure
                         con = consts[i]
-                        # Set override to shear connections in connection
-                        if con.type != 'GENERIC':  # For Point and Fixed costraints
+                        ### For Point and Fixed costraints
+                        ### For Generic constraints
+                        # Tensile constraints - Comment this line out to include all constraints
+                        # Shear constraints - Comment this line out to include all constraints
+                        # Bend constraints - Comment this line out to include all constraints
+                        if con.type != 'GENERIC' \
+                        or con.use_limit_lin_x \
+                        or con.use_limit_lin_y or con.use_limit_lin_z \
+                        or con.use_limit_ang_y or con.use_limit_ang_z:
+                            # Apply breaking threshold incease
                             con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
-                        else:  # For Generic constraints
-                            if con.use_limit_lin_y or con.use_limit_lin_z:  # Shear constraints - Comment this line out to include all constraints
-                                con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
-                            # Set override to bend connections in connection
-                            if con.use_limit_ang_y or con.use_limit_ang_z:  # Bend constraints - Comment this line out to include all constraints
-                                con.breaking_threshold = constsBrkThres[i] +(brkThresInc *rbw_time_scale /rbw_steps_per_second)
-                    
+                
 ################################################################################
 
 def monitor_countIntactConnections_fm(scene):

kk_bullet_constraints_builder/gui.py

@@ -53,7 +53,7 @@
 
 def tool_estimateClusterRadius(scene):
 
-    objs, emptyObjs = gatherObjects(scene)
+    objs, emptyObjs, objsID = gatherObjects(scene)
 
     if len(objs) > 0:        
         print("Estimating optimal cluster radius...")