crates/blenvy/src/blueprints/assets.rs

@@ -58,12 +58,12 @@ pub(crate) struct AssetLoadTracker {
 
 /// helper component, for tracking loaded assets
 #[derive(Component, Debug)]
-pub(crate) struct BlueprintAssetsLoadState {
+pub(crate) struct BlenvyAssetsLoadState {
     pub all_loaded: bool,
     pub asset_infos: Vec<AssetLoadTracker>,
     pub progress: f32,
 }
-impl Default for BlueprintAssetsLoadState {
+impl Default for BlenvyAssetsLoadState {
     fn default() -> Self {
         Self {
             all_loaded: Default::default(),

crates/blenvy/src/blueprints/materials.rs

@@ -17,7 +17,7 @@ use bevy::{
     render::mesh::Mesh,
 };
 
-use crate::{AssetLoadTracker, BlueprintAssetsLoadState, BlenvyConfig, BlueprintInstanceReady};
+use crate::{AssetLoadTracker, BlenvyAssetsLoadState, BlenvyConfig, BlueprintInstanceReady};
 
 #[derive(Component, Reflect, Default, Debug)]
 #[reflect(Component)]
@@ -72,7 +72,7 @@ pub(crate) fn materials_inject(
 
             commands
                 .entity(entity)
-                .insert(BlueprintAssetsLoadState {
+                .insert(BlenvyAssetsLoadState {
                     all_loaded: false,
                     asset_infos,
                     ..Default::default()
@@ -86,7 +86,7 @@ pub(crate) fn materials_inject(
 // TODO, merge with blueprints_check_assets_loading, make generic ?
 pub(crate) fn check_for_material_loaded(
     mut blueprint_assets_to_load: Query<
-        (Entity, &mut BlueprintAssetsLoadState),
+        (Entity, &mut BlenvyAssetsLoadState),
         With<BlueprintMaterialAssetsNotLoaded>,
     >,
     asset_server: Res<AssetServer>,

crates/blenvy/src/blueprints/spawn_from_blueprints.rs

@@ -1,9 +1,9 @@
 use std::path::{Path, PathBuf};
 
-use bevy::{asset::LoadedUntypedAsset, gltf::Gltf, prelude::*, render::view::visibility, scene::SceneInstance, transform::commands, utils::hashbrown::HashMap};
+use bevy::{asset::LoadedUntypedAsset, gltf::Gltf, prelude::*, scene::SceneInstance, transform::commands, utils::hashbrown::HashMap};
 use serde_json::Value;
 
-use crate::{BlueprintAssets, BlueprintAssetsLoadState, AssetLoadTracker, BlenvyConfig, BlueprintAnimations, BlueprintAssetsLoaded, BlueprintAssetsNotLoaded};
+use crate::{BlueprintAssets, BlenvyAssetsLoadState, AssetLoadTracker, BlenvyConfig, BlueprintAnimations, BlueprintAssetsLoaded, BlueprintAssetsNotLoaded};
 
 /// this is a flag component for our levels/game world
 #[derive(Component)]
@@ -164,13 +164,12 @@ asset_server: Res<AssetServer>,
         if !asset_infos.is_empty() {
             commands
                 .entity(entity)
-                .insert(BlueprintAssetsLoadState {
+                .insert(BlenvyAssetsLoadState {
                     all_loaded: false,
                     asset_infos,
                     ..Default::default()
                 })
-                .insert(BlueprintAssetsNotLoaded)                
+                .insert(BlueprintAssetsNotLoaded);
-                ;
         } else {
             commands.entity(entity).insert(BlueprintAssetsLoaded);
         }
@@ -179,7 +178,7 @@ asset_server: Res<AssetServer>,
 
 pub(crate) fn blueprints_check_assets_loading(
     mut blueprint_assets_to_load: Query<
-        (Entity, Option<&Name>, &BlueprintInfo, &mut BlueprintAssetsLoadState),
+        (Entity, Option<&Name>, &BlueprintInfo, &mut BlenvyAssetsLoadState),
         With<BlueprintAssetsNotLoaded>,
     >,
     asset_server: Res<AssetServer>,
@@ -223,7 +222,7 @@ pub(crate) fn blueprints_check_assets_loading(
                 .entity(entity)
                 .insert(BlueprintAssetsLoaded)
                 .remove::<BlueprintAssetsNotLoaded>()
-                //.remove::<BlueprintAssetsLoadState>() //REMOVE it in release mode/ when hot reload is off, keep it for dev/hot reload
+                //.remove::<BlenvyAssetsLoadState>() //REMOVE it in release mode/ when hot reload is off, keep it for dev/hot reload
                 ;
         }else {
             println!("LOADING: done for ALL assets of {:?} (instance of {}): {} ",entity_name, blueprint_info.path, progress * 100.0);
@@ -234,7 +233,7 @@ pub(crate) fn blueprints_check_assets_loading(
 /* 
 pub(crate) fn hot_reload_asset_check(
     mut blueprint_assets: Query<
-    (Entity, Option<&Name>, &BlueprintInfo, &mut BlueprintAssetsLoadState)>,
+    (Entity, Option<&Name>, &BlueprintInfo, &mut BlenvyAssetsLoadState)>,
     asset_server: Res<AssetServer>,
     mut commands: Commands,
 ){
@@ -261,42 +260,109 @@ use bevy::asset::AssetEvent;
 pub(crate) fn react_to_asset_changes(
     mut gltf_events: EventReader<AssetEvent<Gltf>>,
     mut untyped_events: EventReader<AssetEvent<LoadedUntypedAsset>>,
-    mut blueprint_assets: Query<(Entity, Option<&Name>, &BlueprintInfo, &mut BlueprintAssetsLoadState, Option<&Children>)>,
+    mut blueprint_assets: Query<(Entity, Option<&Name>, &BlueprintInfo, &mut BlenvyAssetsLoadState, Option<&Children>)>,
     asset_server: Res<AssetServer>,
     mut commands: Commands,
 
 ) {
 
-    for event in gltf_events.read() {
+    for event in untyped_events.read() {
-        // LoadedUntypedAsset
+        for (entity, entity_name, blueprint_info, mut assets_to_load, c) in blueprint_assets.iter_mut() {
+            for tracker in assets_to_load.asset_infos.iter_mut() {
+                let asset_id = tracker.id;
+
+                println!("changed {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_modified(asset_id));
+            }
+        }
+
         match event {
+            AssetEvent::Added { id } => {
+                // React to the image being created
+                println!("Added untyped {:?}", asset_server.get_path(*id))
+            }
+            AssetEvent::LoadedWithDependencies { id } => {
+                // React to the image being loaded
+                // after all dependencies
+                println!("Loaded with deps untyped {:?}", asset_server.get_path(*id))
+            }
             AssetEvent::Modified { id } => {
                 // React to the image being modified
-                // println!("Modified gltf {:?}", asset_server.get_path(*id));
+                println!("Modified untyped {:?}", asset_server.get_path(*id))
+            }
+            AssetEvent::Removed { id } => {
+                // React to the image being removed
+                println!("Removed untyped {:?}", asset_server.get_path(*id))
+            },
+            AssetEvent::Unused { id } => {
+                // React to the last strong handle for the asset being dropped
+                println!("unused untyped {:?}", asset_server.get_path(*id))
+            }
+        }
+    }
+    for event in gltf_events.read() {
+        // LoadedUntypedAsset
+
+        /*for (entity, entity_name, blueprint_info, mut assets_to_load) in blueprint_assets.iter_mut() {
+            for tracker in assets_to_load.asset_infos.iter_mut() {
+                let asset_id = tracker.id;
+                if blueprint_info.path.ends_with("glb") || blueprint_info.path.ends_with("gltf") {
+                    // let typed_asset_id = asset_server.get_handle(blueprint_info.path);
+                    let foo: Handle<Gltf> = asset_server.load(blueprint_info.path.clone());
+                    //println!("changed {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_modified(foo.id()));
+                    println!("changed {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_modified(foo.id()));
+                    println!("added {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_added(foo.id()));
+                    println!("removed {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_removed(foo.id()));
+                    println!("loaded with deps {:?} (blueprint {}) {}", entity_name, blueprint_info.path, event.is_loaded_with_dependencies(foo.id()));
+
+                }
+            }
+        }*/
+
+        match event {
+            AssetEvent::Added { id } => {
+                // React to the image being created
+                println!("Added gltf, path {:?}", asset_server.get_path(*id));
+                
+            }
+            AssetEvent::LoadedWithDependencies { id } => {
+                // React to the image being loaded
+                // after all dependencies
+                println!("Loaded gltf with deps{:?}", asset_server.get_path(*id))
+            }
+            AssetEvent::Modified { id } => {
+                // React to the image being modified
+                println!("Modified gltf {:?}", asset_server.get_path(*id));
+
                 for (entity, entity_name, blueprint_info, mut assets_to_load, children) in blueprint_assets.iter_mut() {
                     for tracker in assets_to_load.asset_infos.iter_mut() {
-                        if asset_server.get_path(*id).is_some() {
+                        if tracker.path == asset_server.get_path(*id).unwrap().to_string() {
-                            if tracker.path == asset_server.get_path(*id).unwrap().to_string() {
+                            println!("HOLY MOLY IT DETECTS !!, now respawn {:?}", entity_name);
-                                println!("HOLY MOLY IT DETECTS !!, now respawn {:?}", entity_name);
+                            if children.is_some() {
-                                if children.is_some() {
+                                for child in children.unwrap().iter(){
-                                    for child in children.unwrap().iter(){
+                                    commands.entity(*child).despawn_recursive();
-                                        commands.entity(*child).despawn_recursive();
-                                    }
-                                }
-                                commands.entity(entity)
-                                    .remove::<BlueprintAssetsLoaded>()
-                                    .remove::<SceneInstance>()
-                                    .remove::<BlueprintAssetsLoadState>()
-                                    .insert(SpawnHere);
 
-                                break;
+                                }
                             }
+                            commands.entity(entity)
+                            .remove::<BlueprintAssetsLoaded>()
+                            .remove::<SceneInstance>()
+                            .remove::<BlenvyAssetsLoadState>()
+                            .insert(SpawnHere);
+
+                            break;
                         }
                     }
                 }
 
             }
-            _ => {}
+            AssetEvent::Removed { id } => {
+                // React to the image being removed
+                println!("Removed gltf {:?}", asset_server.get_path(*id))
+            },
+            AssetEvent::Unused { id } => {
+                // React to the last strong handle for the asset being dropped
+                println!("unused gltf {:?}", asset_server.get_path(*id))
+            }
         }
     }
 }
@@ -381,8 +447,6 @@ pub(crate) fn blueprints_spawn(
             SceneBundle {
                 scene: scene.clone(),
                 transform: transforms,
-                visibility: Visibility::Hidden,
-
                 ..Default::default()
             },
             Spawned,

crates/blenvy/src/blueprints/spawn_post_process.rs

@@ -99,11 +99,10 @@ pub(crate) fn spawned_blueprint_post_process(
         commands.entity(original).remove::<SpawnHere>();
         commands.entity(original).remove::<Spawned>();
         // commands.entity(original).remove::<Handle<Scene>>(); // FIXME: if we delete the handle to the scene, things get despawned ! not what we want
-        //commands.entity(original).remove::<BlueprintAssetsLoadState>(); // also clear the sub assets tracker to free up handles, perhaps just freeing up the handles and leave the rest would be better ?
+        //commands.entity(original).remove::<BlenvyAssetsLoadState>(); // also clear the sub assets tracker to free up handles, perhaps just freeing up the handles and leave the rest would be better ?
         //commands.entity(original).remove::<BlueprintAssetsLoaded>();
         commands.entity(root_entity).despawn_recursive();
-        commands.entity(original).insert(            Visibility::Visible
+
-        );
         blueprint_events.send(BlueprintEvent::Spawned {blueprint_name: blueprint_info.name.clone(), blueprint_path: blueprint_info.path.clone() });
         
         debug!("DONE WITH POST PROCESS");

testing/bevy_example/src/game/in_game.rs

@@ -62,7 +62,7 @@ pub fn spawn_test(
         let new_entity = commands
             .spawn((
                 BluePrintBundle {
-                    blueprint: BlueprintInfo{name: "Blueprint1".into() , path:"blueprints/Blueprint1.glb".into()}, // FIXME
+                    blueprint: BlueprintInfo{name: "Health_Pickup".into() , path:"foo/bar.glb".into()}, // FIXME
                     ..Default::default()
                 },
                 bevy::prelude::Name::from(format!("test{}", name_index)),

tools/blenvy/TODO.md

@@ -174,17 +174,17 @@ Blender side:
 - [x] prevent attempting to add unexisting components to targets (ie when using the component search)
     - [x] also for the bulk fix actions
 - [x] selection of nested objects in collections IS NOT WORKING !!! AHH
-- [x] fix/ overhaul upgreadable components
+- [ ] fix/ overhaul upgreadable components
     - [x] add listing of upgradeable components for
         - [x] meshes
         - [x] materials
     - [x] fix display of upgradeaeble components & co
     - [x] add clear visual distinction between internal (selectable) & non selectable ones
     - [x] do not make selection button available for external blueprints/collections 
-        - [x] perhaps do not show the other buttons & inputs either ? we cannot change the values of an external library file anyway
+        - [ ] perhaps do not show the other buttons & inputs either ? we cannot change the values of an external library file anyway
 
-- [x] BLENVY_OT_item_select is missing handling for the other types (outside of object & collection)
+- [ ] BLENVY_OT_item_select is missing handling for the other types (outside of object & collection)
-    - [x] fix selection logic 
+        - [ ] fix selection logic 
 
 - [ ] hidden objects/collections only semi respected at export
     - this is because blueprints are external ?
@@ -209,9 +209,6 @@ Blender side:
     - [ ] scan for used materials per blueprint !
     - [ ] for scenes, scan for used materials of all non instance objects (TODO: what about overrides ?)
 
-- [ ] add a way of visualizing per blueprint instances ?
-- [ ] display export path of blueprints (mostly external) ?
-
 Bevy Side:
 - [x] deprecate BlueprintName & BlueprintPath & use BlueprintInfo instead
 - [ ] make blueprint instances invisible until spawning is done to avoid "spawn flash"?
@@ -219,22 +216,14 @@ Bevy Side:
 - [ ] simplify testing example:
     - [x] remove use of rapier physics (or even the whole common boilerplate ?)
     - [ ] remove/replace bevy editor pls with some native ui to display hierarchies
-    - [ ] a full fledged demo (including physics & co)
+- [x] try out hot reloading
-    - [ ] other examples without interactions or physics 
+- [ ] add a way of overriding assets for collection instances
-- [ ] add hot reloading
+    - [ ] add a way of visualizing per blueprint instances
-    - [x] basics
-    - [ ] make it enabled/disabled based on general flag
-    - [ ] cleanup internals
-- [ ] review & change general component insertion & spawning ordering & logic
-    - GltfComponentsSet::Injection => GltfBlueprintsSet::Spawn => GltfBlueprintsSet::AfterSpawn
-        Injection => inject lights & co => spawn => afterSpawn
-                                                 => Injection => inject lights & co 
-
-- [ ] add a way of overriding assets for collection instances => doubt this is possible
 - [ ] cleanup all the spurious debug messages
 - [ ] fix animation handling
-    - [ ] how to deal with animation graphs ?
+- [ ] simplify examples:
-
+    - [ ] a full fledged demo (including physics & co)
+    - [ ] other examples without interactions or physics 
 - [ ] update main docs
     - [ ] rename project to Blenvy
     - [ ] replace all references to the old 2 add-ons with those to Blenvy

tools/blenvy/add_ons/bevy_components/components/ui.py

@@ -339,15 +339,17 @@ class BLENVY_PT_component_tools_panel(bpy.types.Panel):
         registry = bpy.context.window_manager.components_registry 
         registry_has_type_infos = registry.has_type_infos()
         selected_component = target.components_meta.component_selector
-        internal = target.library is None
+
         row = layout.row()
 
         col = row.column()
         selector_text = f"{target.name}({item_type_short})"
-        operator = col.operator("blenvy.select_item", text=selector_text, icon="FILE_BLEND")
+        if target.library is None:
-        operator.target_name = target.name
+            operator = col.operator("blenvy.select_item", text=selector_text, icon="FILE_BLEND")
-        operator.item_type = item_type
+            operator.target_name = target.name
-        col.enabled = internal
+            operator.item_type = item_type
+        else:
+            col.label(text=selector_text, icon="LIBRARY_DATA_DIRECT")
 
         col = row.column()
         col.label(text=status)
@@ -356,12 +358,9 @@ class BLENVY_PT_component_tools_panel(bpy.types.Panel):
         col.label(text=component_name)
 
         col = row.column()
-        if internal:
+        # each components_meta has a component selector to pick components from
-            # each components_meta has a component selector to pick components from
+        components_meta = target.components_meta
-            components_meta = target.components_meta
+        col.prop(components_meta, "component_selector", text="")
-            col.prop(components_meta, "component_selector", text="")
-        else:
-            col.label(text="external, cannot edit")
 
 
         col = row.column()
@@ -370,7 +369,7 @@ class BLENVY_PT_component_tools_panel(bpy.types.Panel):
         operator.original_name = component_name
         operator.target_items = json.dumps([(target.name, item_type)]) # tupple
         operator.target_name = target_component_name
-        col.enabled = internal and registry_has_type_infos and component_name != "" and target_component_name != ""  and component_name != target_component_name
+        col.enabled = registry_has_type_infos and component_name != "" and target_component_name != ""  and component_name != target_component_name
 
 
         col = row.column()
@@ -378,7 +377,6 @@ class BLENVY_PT_component_tools_panel(bpy.types.Panel):
         operator.item_name = target.name
         operator.component_name = component_name
         operator.item_type = get_selection_type(target)
-        col.enabled = internal
 
     def draw_invalid_items(self, layout, upgreadable_entries):
         for entry in upgreadable_entries:

tools/blenvy/add_ons/bevy_components/utils.py

@@ -10,7 +10,7 @@ def select_area(context, area_name):
     for area in context.screen.areas:
         #if area.type == 'PROPERTIES' and context.object is not None and context.object.type not in ('LIGHT_PROBE', 'CAMERA', 'LIGHT', 'SPEAKER'):
         # Set it the active space
-        #print("SELECT AREA", area_name)
+        print("SELECT AREA", area_name)
         try:
             area.spaces.active.context = area_name #'MATERIAL' # 'VIEW_LAYER', 'SCENE' etc.
         except Exception as error: 
@@ -103,6 +103,7 @@ class BLENVY_OT_item_select(Operator):
                     bpy.ops.object.select_all(action='DESELECT')
                     bpy.context.window.scene = scene_of_collection
                     #bpy.context.view_layer.objects.active = None
+                    #
                     context.window_manager.blenvy_item_selected_ids = json.dumps({"name": collection.name, "type": self.item_type})
                     set_active_collection(bpy.context.window.scene, collection.name)
 
@@ -115,6 +116,7 @@ class BLENVY_OT_item_select(Operator):
                 if scene_of_item is not None:
                     bpy.ops.object.select_all(action='DESELECT')
                     bpy.context.window.scene = scene_of_item
+                    mesh_object.select_set(True)    
                     bpy.context.view_layer.objects.active = mesh_object
 
                     context.window_manager.blenvy_item_selected_ids = json.dumps({"name": mesh.name, "type": self.item_type})
@@ -126,9 +128,12 @@ class BLENVY_OT_item_select(Operator):
                 material_object = get_material_object(material)
                 scene_of_item = get_object_scene(material_object)
                 select_area(context=context, area_name="MATERIAL")
+                print("scene_of_item", scene_of_item)
                 if scene_of_item is not None:
+
                     bpy.ops.object.select_all(action='DESELECT')
                     bpy.context.window.scene = scene_of_item
+                    #material_object.select_set(True)    
                     bpy.context.view_layer.objects.active = material_object
 
                     context.window_manager.blenvy_item_selected_ids = json.dumps({"name": material.name, "type": self.item_type})
@@ -149,6 +154,20 @@ def get_selected_item(context):
     #print("original context", context)
    
 
+    try:
+        selection_overrides = json.loads(context.window_manager.blenvy_item_selected_ids)
+        #print("selection_overrides", selection_overrides)
+        if selection_overrides["type"] == "OBJECT":
+            selection = bpy.data.objects[selection_overrides["name"]]
+        elif selection_overrides["type"] == "COLLECTION":
+            selection = bpy.data.collections[selection_overrides["name"]]
+        if selection_overrides["type"] == "MESH":
+            selection = bpy.data.meshes[selection_overrides["name"]]
+        elif selection_overrides["type"] == "MATERIAL":
+            selection = bpy.data.materials[selection_overrides["name"]]
+        #print("SELECTION", selection)
+        #context.window_manager.blenvy_item_selected_ids = "{}"
+    except: pass
 
     if selection is None:
         area = get_outliner_area()
@@ -156,8 +175,8 @@ def get_selected_item(context):
             region = next(region for region in area.regions if region.type == "WINDOW")
             with bpy.context.temp_override(area=area, region=region):
                 #print("overriden context", bpy.context)
-                """for obj in bpy.context.selected_ids:
+                for obj in bpy.context.selected_ids:
-                    print(f"Selected: {obj.name} - {type(obj)}")"""
+                    print(f"Selected: {obj.name} - {type(obj)}")
                 number_of_selections = len(bpy.context.selected_ids)
                 selection = bpy.context.selected_ids[number_of_selections - 1] if number_of_selections > 0 else None #next(iter(bpy.context.selected_ids), None)
 
@@ -166,22 +185,6 @@ def get_selected_item(context):
     if selection is None:
         number_of_selections = len(context.selected_objects)
         selection = context.selected_objects[number_of_selections - 1] if number_of_selections > 0 else None
-
-    if selection is None:
-        try:
-            selection_overrides = json.loads(context.window_manager.blenvy_item_selected_ids)
-            #print("selection_overrides", selection_overrides)
-            if selection_overrides["type"] == "OBJECT":
-                selection = bpy.data.objects[selection_overrides["name"]]
-            elif selection_overrides["type"] == "COLLECTION":
-                selection = bpy.data.collections[selection_overrides["name"]]
-            if selection_overrides["type"] == "MESH":
-                selection = bpy.data.meshes[selection_overrides["name"]]
-            elif selection_overrides["type"] == "MATERIAL":
-                selection = bpy.data.materials[selection_overrides["name"]]
-            #print("SELECTION", selection)
-            #context.window_manager.blenvy_item_selected_ids = "{}"
-        except: pass
     return selection