feat(blenvy): continued work on auto_export refactor

tools/blenvy/TODO.md

@@ -48,12 +48,17 @@ Blueprints:
 
     - [ ] scan & inject on load
     - [ ] scan & inject on save
+    - [ ] decide where & when to do & store blueprints data
 
 General things to solve:
  - [x] save settings
  - [x] load settings
     - [ ] add_blueprints_data
 
+- [ ] rename all path stuff using the old naming convention : "export_blueprints_path_full"
+- [ ] generate the full paths directly when setting them in the UI  
+    - [ ] problem : how to deal with defaults: do it on start/load ?
+
 General issues:
  - there is no safeguard for naming collisions for naming across blender files
  - this can cause an issue for assets list "parent" 

tools/blenvy/blueprints/blueprints_scan.py

@@ -8,7 +8,7 @@ from .blueprint import Blueprint
 # - with the "auto_export" flag
 # https://blender.stackexchange.com/questions/167878/how-to-get-all-collections-of-the-current-scene
 def blueprints_scan(main_scenes, library_scenes, addon_prefs):
-    export_marked_assets = getattr(addon_prefs,"export_marked_assets")
+    export_marked_assets = getattr(addon_prefs.auto_export,"export_marked_assets")
 
     blueprints = {}
     blueprints_from_objects = {}

tools/blenvy/core/blenvy_manager.py

@@ -7,6 +7,8 @@ import blenvy.gltf_auto_export.settings as auto_export_settings
 
 def update_scene_lists(self, context):                
     blenvy = self# context.window_manager.blenvy
+    blenvy.main_scene_names = [scene.name for scene in blenvy.main_scenes] # FIXME: unsure
+    blenvy.library_scene_names = [scene.name for scene in blenvy.library_scenes] # FIXME: unsure
     upsert_settings(blenvy.settings_save_path, {"common_main_scene_names": [scene.name for scene in blenvy.main_scenes]})
     upsert_settings(blenvy.settings_save_path, {"common_library_scene_names": [scene.name for scene in blenvy.library_scenes]})
 
@@ -74,13 +76,15 @@ class BlenvyManager(PropertyGroup):
 
     main_scenes: CollectionProperty(name="main scenes", type=SceneSelector) # type: ignore
     main_scenes_index: IntProperty(name = "Index for main scenes list", default = 0, update=update_scene_lists) # type: ignore
+    main_scene_names = [] # FIXME: unsure
 
     library_scenes: CollectionProperty(name="library scenes", type=SceneSelector) # type: ignore
     library_scenes_index: IntProperty(name = "Index for library scenes list", default = 0, update=update_scene_lists) # type: ignore
+    library_scene_names = [] # FIXME: unsure
 
     # sub ones
     auto_export: PointerProperty(type=auto_export_settings.AutoExportSettings) # type: ignore
-    #components: PointerProperty(type=auto_export_settings.AutoExportSettings) # type: ignore
+    #components: PointerProperty(type=bevy_component_settings.ComponentSettings) # type: ignore
 
     def is_scene_ok(self, scene):
         try:

tools/blenvy/gltf_auto_export/auto_export/auto_export.py

@@ -28,16 +28,16 @@ def auto_export(changes_per_scene, changed_export_parameters, addon_prefs):
         # Get the folder
         blend_file_path = os.path.dirname(file_path)
         
+        print("settings", dict(addon_prefs))
         # get the preferences for our addon
         project_root_path = getattr(addon_prefs, "project_root_path")
         assets_path = getattr(addon_prefs,"assets_path")
 
         #should we use change detection or not 
-        export_change_detection = getattr(addon_prefs, "export_change_detection")
-        export_scene_settings = getattr(addon_prefs,"export_scene_settings")
-
-        do_export_blueprints = getattr(addon_prefs,"export_blueprints")
-        export_materials_library = getattr(addon_prefs,"export_materials_library")
+        change_detection = getattr(addon_prefs.auto_export, "change_detection")
+        export_scene_settings = getattr(addon_prefs.auto_export,"export_scene_settings")
+        do_export_blueprints = getattr(addon_prefs.auto_export,"export_blueprints")
+        export_materials_library = getattr(addon_prefs.auto_export,"export_materials_library")
         print("export_materials_library", export_materials_library)
 
         # standard gltf export settings are stored differently
@@ -45,21 +45,6 @@ def auto_export(changes_per_scene, changed_export_parameters, addon_prefs):
         gltf_extension = standard_gltf_exporter_settings.get("export_format", 'GLB')
         gltf_extension = '.glb' if gltf_extension == 'GLB' else '.gltf'
        
-        # here we do a bit of workaround by creating an override # TODO: do this at the "UI" level
-        print("collection_instances_combine_mode", addon_prefs.collection_instances_combine_mode)
-        """if hasattr(addon_prefs, "__annotations__") :
-            tmp = {}
-            for k in AutoExportGltfAddonPreferences.__annotations__:
-                item = AutoExportGltfAddonPreferences.__annotations__[k]
-                #print("tutu",k, item.keywords.get('default', None) )
-                default = item.keywords.get('default', None)
-                tmp[k] = default
-            
-            for (k, v) in addon_prefs.properties.items():
-                tmp[k] = v
-
-            addon_prefs = SimpleNamespace(**tmp) #copy.deepcopy(addon_prefs)
-            addon_prefs.__annotations__ = tmp"""
         # generate the actual complete output paths
         addon_prefs.export_assets_path_full = os.path.join(blend_file_path, project_root_path, assets_path)
         addon_prefs.export_blueprints_path_full = os.path.join(addon_prefs.export_assets_path_full, getattr(addon_prefs,"blueprints_path"))
@@ -80,7 +65,7 @@ def auto_export(changes_per_scene, changed_export_parameters, addon_prefs):
         inject_export_path_into_internal_blueprints(internal_blueprints=blueprints_data.internal_blueprints, blueprints_path=blueprints_path, gltf_extension=gltf_extension)
         for blueprint in blueprints_data.blueprints:
             bpy.context.window_manager.blueprints_registry.add_blueprint(blueprint)
-        bpy.context.window_manager.blueprints_registry.add_blueprints_data()
+        #bpy.context.window_manager.blueprints_registry.add_blueprints_data()
 
         if export_scene_settings:
             # inject/ update scene components
@@ -92,7 +77,7 @@ def auto_export(changes_per_scene, changed_export_parameters, addon_prefs):
 
         # export
         if do_export_blueprints:
-            print("EXPORTING")
+            print("EXPORTING", blueprints_data)
             # get blueprints/collections infos
             (blueprints_to_export) = get_blueprints_to_export(changes_per_scene, changed_export_parameters, blueprints_data, addon_prefs)
              
@@ -137,7 +122,7 @@ def auto_export(changes_per_scene, changed_export_parameters, addon_prefs):
                     export_main_scene(bpy.data.scenes[scene_name], blend_file_path, addon_prefs, blueprints_data)
 
             # now deal with blueprints/collections
-            do_export_library_scene = not export_change_detection or changed_export_parameters or len(blueprints_to_export) > 0
+            do_export_library_scene = not change_detection or changed_export_parameters or len(blueprints_to_export) > 0
             if do_export_library_scene:
                 print("export LIBRARY")
                 export_blueprints(blueprints_to_export, addon_prefs, blueprints_data)

tools/blenvy/gltf_auto_export/auto_export/export_blueprints.py

@@ -8,13 +8,13 @@ from ..helpers.helpers_scenes import clear_hollow_scene, copy_hollowed_collectio
 
    
 def export_blueprints(blueprints, addon_prefs, blueprints_data):
-    export_blueprints_path_full = getattr(addon_prefs,"export_blueprints_path_full")
+    export_blueprints_path_full = getattr(addon_prefs, "export_blueprints_path_full")
     gltf_export_preferences = generate_gltf_export_preferences(addon_prefs)
     
     try:
         # save current active collection
         active_collection =  bpy.context.view_layer.active_layer_collection
-        export_materials_library = getattr(addon_prefs,"export_materials_library")
+        export_materials_library = getattr(addon_prefs.auto_export, "export_materials_library")
 
         for blueprint in blueprints:
             print("exporting collection", blueprint.name)

tools/blenvy/gltf_auto_export/auto_export/export_main_scenes.py

@@ -14,8 +14,8 @@ def export_main_scene(scene, blend_file_path, addon_prefs, blueprints_data):
     export_assets_path_full = getattr(addon_prefs,"export_assets_path_full")
     export_levels_path_full = getattr(addon_prefs,"export_levels_path_full")
 
-    export_blueprints = getattr(addon_prefs,"export_blueprints")
-    export_separate_dynamic_and_static_objects = getattr(addon_prefs, "export_separate_dynamic_and_static_objects")
+    export_blueprints = getattr(addon_prefs.auto_export,"export_blueprints")
+    export_separate_dynamic_and_static_objects = getattr(addon_prefs.auto_export, "export_separate_dynamic_and_static_objects")
 
     export_settings = { **gltf_export_preferences, 
                        'use_active_scene': True, 

tools/blenvy/gltf_auto_export/auto_export/get_blueprints_to_export.py

@@ -5,20 +5,20 @@ from ...blueprints.blueprint_helpers import find_blueprints_not_on_disk
 
 # TODO: this should also take the split/embed mode into account: if a nested collection changes AND embed is active, its container collection should also be exported
 def get_blueprints_to_export(changes_per_scene, changed_export_parameters, blueprints_data, addon_prefs):
-    export_change_detection = getattr(addon_prefs, "export_change_detection")
     export_gltf_extension = getattr(addon_prefs, "export_gltf_extension", ".glb")
     export_blueprints_path_full = getattr(addon_prefs,"export_blueprints_path_full", "")
-    collection_instances_combine_mode = getattr(addon_prefs, "collection_instances_combine_mode")
+    change_detection = getattr(addon_prefs.auto_export, "change_detection")
+    collection_instances_combine_mode = getattr(addon_prefs.auto_export, "collection_instances_combine_mode")
 
     [main_scene_names, level_scenes, library_scene_names, library_scenes] = get_scenes(addon_prefs)
     internal_blueprints = blueprints_data.internal_blueprints
     blueprints_to_export = internal_blueprints # just for clarity
 
-    # print("export_change_detection", export_change_detection, "changed_export_parameters", changed_export_parameters, "changes_per_scene", changes_per_scene)
+    # print("change_detection", change_detection, "changed_export_parameters", changed_export_parameters, "changes_per_scene", changes_per_scene)
     
     # if the export parameters have changed, bail out early
     # we need to re_export everything if the export parameters have been changed
-    if export_change_detection and not changed_export_parameters:
+    if change_detection and not changed_export_parameters:
         changed_blueprints = []
 
         # first check if all collections have already been exported before (if this is the first time the exporter is run

tools/blenvy/gltf_auto_export/auto_export/get_levels_to_export.py

@@ -37,15 +37,16 @@ def changed_object_in_scene(scene_name, changes_per_scene, blueprints_data, coll
 
 # this also takes the split/embed mode into account: if a collection instance changes AND embed is active, its container level/world should also be exported
 def get_levels_to_export(changes_per_scene, changed_export_parameters, blueprints_data, addon_prefs):
-    export_change_detection = getattr(addon_prefs, "export_change_detection")
     export_gltf_extension = getattr(addon_prefs, "export_gltf_extension")
     export_levels_path_full = getattr(addon_prefs, "export_levels_path_full")
-    collection_instances_combine_mode = getattr(addon_prefs, "collection_instances_combine_mode")
+
+    change_detection = getattr(addon_prefs.auto_export, "change_detection")
+    collection_instances_combine_mode = getattr(addon_prefs.auto_export, "collection_instances_combine_mode")
 
     [main_scene_names, level_scenes, library_scene_names, library_scenes] = get_scenes(addon_prefs)
  
     # determine list of main scenes to export
     # we have more relaxed rules to determine if the main scenes have changed : any change is ok, (allows easier handling of changes, render settings etc)
-    main_scenes_to_export = [scene_name for scene_name in main_scene_names if not export_change_detection or changed_export_parameters or scene_name in changes_per_scene.keys() or changed_object_in_scene(scene_name, changes_per_scene, blueprints_data, collection_instances_combine_mode) or not check_if_blueprint_on_disk(scene_name, export_levels_path_full, export_gltf_extension) ]
+    main_scenes_to_export = [scene_name for scene_name in main_scene_names if not change_detection or changed_export_parameters or scene_name in changes_per_scene.keys() or changed_object_in_scene(scene_name, changes_per_scene, blueprints_data, collection_instances_combine_mode) or not check_if_blueprint_on_disk(scene_name, export_levels_path_full, export_gltf_extension) ]
 
     return (main_scenes_to_export)

tools/blenvy/gltf_auto_export/auto_export/operators.py

@@ -22,7 +22,7 @@ class AutoExportGLTF(Operator, AutoExportGltfAddonPreferences):#, ExportHelper):
         'auto_export',
         'project_root_path',
         'assets_path',
-        'export_change_detection',
+        'change_detection',
         'export_scene_settings',
 
         'main_scene_names',
@@ -132,6 +132,7 @@ class AutoExportGLTF(Operator, AutoExportGltfAddonPreferences):#, ExportHelper):
     This should ONLY be run when actually doing exports/aka calling auto_export function, because we only care about the difference in settings between EXPORTS
     """
     def did_export_settings_change(self):
+        return True
         # compare both the auto export settings & the gltf settings
         previous_auto_settings = bpy.data.texts[".gltf_auto_export_settings_previous"] if ".gltf_auto_export_settings_previous" in bpy.data.texts else None
         previous_gltf_settings = bpy.data.texts[".gltf_auto_export_gltf_settings_previous"] if ".gltf_auto_export_gltf_settings_previous" in bpy.data.texts else None
@@ -183,39 +184,41 @@ class AutoExportGLTF(Operator, AutoExportGltfAddonPreferences):#, ExportHelper):
         return changed
     
     def did_objects_change(self):
-        pass
+        # FIXME: add it back
+        return {}
 
     def execute(self, context):
+        print("execute auto export", context.window_manager.blenvy.auto_export) 
+        blenvy = context.window_manager.blenvy
+        auto_export_settings = blenvy.auto_export
         bpy.context.window_manager.auto_export_tracker.disable_change_detection()
         if self.direct_mode:
             self.load_settings(context)
         if self.will_save_settings:
             self.save_settings(context)
         #print("self", self.auto_export)
-        if self.auto_export: # only do the actual exporting if auto export is actually enabled
+        if auto_export_settings.auto_export: # only do the actual exporting if auto export is actually enabled
+            print("auto export")
             #changes_per_scene = context.window_manager.auto_export_tracker.changed_objects_per_scene
             #& do the export
-            if self.direct_mode: #Do not auto export when applying settings in the menu, do it on save only   
-                # determine changed objects
-                changes_per_scene = self.did_objects_change()
-                # determine changed parameters 
-                params_changed = self.did_export_settings_change()
-                auto_export(changes_per_scene, params_changed, self)
-                # cleanup 
-                # reset the list of changes in the tracker
-                bpy.context.window_manager.auto_export_tracker.clear_changes()
-                print("AUTO EXPORT DONE")            
+            # determine changed objects
+            changes_per_scene = self.did_objects_change()
+            # determine changed parameters 
+            params_changed = self.did_export_settings_change()
+            auto_export(changes_per_scene, params_changed, blenvy)
+            # cleanup 
+            # reset the list of changes in the tracker
+            bpy.context.window_manager.auto_export_tracker.clear_changes()
+            print("AUTO EXPORT DONE")            
             bpy.app.timers.register(bpy.context.window_manager.auto_export_tracker.enable_change_detection, first_interval=0.1)
         else: 
             print("auto export disabled, skipping")
         return {'FINISHED'}    
     
     def invoke(self, context, event):
-        #print("invoke")
+        print("invoke")
         bpy.context.window_manager.auto_export_tracker.disable_change_detection()
         self.load_settings(context)
-        wm = context.window_manager
-        #wm.fileselect_add(self)
         return context.window_manager.invoke_props_dialog(self, title="Auto export", width=640)
     
     def cancel(self, context):

tools/blenvy/gltf_auto_export/auto_export/preferences.py

@@ -19,7 +19,7 @@ AutoExportGltfPreferenceNames = [
     'export_scene_settings',
 
     'show_change_detection_settings',
-    'export_change_detection',
+    'change_detection',
 
     'show_scene_settings',
     'main_scenes',
@@ -104,7 +104,7 @@ class AutoExportGltfAddonPreferences(AddonPreferences):
         default=True
     ) # type: ignore 
 
-    export_change_detection: BoolProperty(
+    change_detection: BoolProperty(
         name='Change detection',
         description='Use change detection to determine what/if should be exported',
         default=True

tools/blenvy/gltf_auto_export/helpers/helpers_scenes.py

@@ -146,7 +146,7 @@ def duplicate_object(object, parent, combine_mode, destination_collection, bluep
 
 # copies the contents of a collection into another one while replacing library instances with empties
 def copy_hollowed_collection_into(source_collection, destination_collection, parent_empty=None, filter=None, blueprints_data=None, addon_prefs={}):
-    collection_instances_combine_mode = getattr(addon_prefs, "collection_instances_combine_mode")
+    collection_instances_combine_mode = getattr(addon_prefs.auto_export, "collection_instances_combine_mode")
 
     for object in source_collection.objects:
         if object.name.endswith("____bak"): # some objects could already have been handled, ignore them
@@ -207,9 +207,13 @@ def clear_hollow_scene(temp_scene, original_root_collection):
 
 # convenience utility to get lists of scenes
 def get_scenes(addon_prefs):
-    level_scene_names= getattr(addon_prefs,"main_scene_names", []) #list(map(lambda scene: scene.name, getattr(addon_prefs,"main_scenes")))
-    library_scene_names = getattr(addon_prefs,"library_scene_names", []) #list(map(lambda scene: scene.name, getattr(addon_prefs,"library_scenes")))
+    level_scene_names= getattr(addon_prefs, "main_scene_names", []) #list(map(lambda scene: scene.name, getattr(addon_prefs,"main_scenes")))
+    library_scene_names = getattr(addon_prefs, "library_scene_names", []) #list(map(lambda scene: scene.name, getattr(addon_prefs,"library_scenes")))
 
+    level_scene_names= list(map(lambda scene: scene.name, getattr(addon_prefs,"main_scenes")))
+    library_scene_names = list(map(lambda scene: scene.name, getattr(addon_prefs,"library_scenes")))
+
+    print("level_scene_names", level_scene_names)
     level_scene_names = list(filter(lambda name: name in bpy.data.scenes, level_scene_names))
     library_scene_names = list(filter(lambda name: name in bpy.data.scenes, library_scene_names))
 

tools/blenvy/gltf_auto_export/settings.py

@@ -3,11 +3,6 @@ from bpy_types import (PropertyGroup)
 from bpy.props import (EnumProperty, PointerProperty, StringProperty, BoolProperty, CollectionProperty, IntProperty)
 
 class AutoExportSettings(PropertyGroup):
-    # use when operator is called directly, works a bit differently than inside the ui
-    direct_mode: BoolProperty(
-        default=False
-    ) # type: ignore
-
     auto_export: BoolProperty(
         name='Auto export',
         description='Automatically export to gltf on save',
@@ -15,7 +10,7 @@ class AutoExportSettings(PropertyGroup):
     ) # type: ignore
 
     #### general
-    export_change_detection: BoolProperty(
+    change_detection: BoolProperty(
         name='Change detection',
         description='Use change detection to determine what/if should be exported',
         default=True

tools/blenvy/gltf_auto_export/ui.py

@@ -29,7 +29,7 @@ def draw_settings_ui(layout, auto_export_settings):
     if panel:
         section = panel.box()
         section.enabled = controls_enabled
-        section.prop(auto_export_settings, "export_change_detection", text="Use change detection")
+        section.prop(auto_export_settings, "change_detection", text="Use change detection")
 
     header, panel = layout.panel("Blueprints", default_closed=False)
     header.label(text="Blueprints")