Make scene loader configurable

Author: einarf

demosys/conf/default_settings.py

@@ -19,6 +19,7 @@
     'TEXTURE_FINDERS',
     'SCENE_DIRS',
     'SCENE_FINDERS',
+    'SCENE_LOADERS',
 )
 
 DEBUG = False
@@ -93,3 +94,8 @@
     "demosys.core.scenefiles.finders.FileSystemFinder",
     "demosys.core.scenefiles.finders.EffectDirectoriesFinder",
 )
+
+SCENE_LOADERS = (
+    'demosys.scene.loaders.gltf.GLTF2',
+    'demosys.scene.loaders.wavefront.ObjLoader',
+)

demosys/resources/scenes.py

@@ -3,6 +3,8 @@
 from demosys.core.exceptions import ImproperlyConfigured
 from demosys.core.scenefiles.finders import get_finders
 from demosys.scene import Scene
+from demosys.conf import settings
+from demosys.utils.module_loading import import_string
 
 
 class Scenes:
@@ -27,10 +29,16 @@ def get(self, path, create=False, **kwargs):
         :param create: (bool) Create an empty scene object if it doesn't exist
         :return: Scene object
         """
-        # FIXME: Figure out what scene loader class should be used instead of hardcoding gltf
-        # FIXME: This should be configured in settings
-        if path not in self.scenes:
-            self.scenes[path] = Scene(path, loader=gltf.GLTF2(path), **kwargs)
+        # Figure out what scene loader class should be used
+        for loader_name in settings.SCENE_LOADERS:
+            loader_cls = import_string(loader_name)
+            if loader_cls.supports_file(path):
+                if path not in self.scenes:
+                    loader_cls = import_string(loader_name)
+                    self.scenes[path] = Scene(path, loader=loader_cls(path), **kwargs)
+                break
+        else:
+            raise ImproperlyConfigured("Scene {} has no loader class registered. Check settings.SCENE_LOADERS")
 
         return self.scenes[path]