fix bug in UI

Author: mam10eks

application/src/tira_app/endpoints/admin_api.py

@@ -17,11 +17,7 @@
 
 from .. import tira_model as model
 from ..authentication import auth
-from ..checks import (
-    check_conditional_permissions,
-    check_permissions,
-    check_resources_exist,
-)
+from ..checks import check_conditional_permissions, check_permissions, check_resources_exist
 from ..git_runner import check_that_git_integration_is_valid
 from ..ir_datasets_loader import run_irds_command
 from .v1._datasets import download_mirrored_resource
@@ -219,18 +215,19 @@ def admin_delete_task(request, task_id):
 def file_listing(path, title):
     path = Path(path)
     children = []
-    for f in os.listdir(path):
-        if len(children) > 5:
-            children += [{"title": "..."}]
-            break
-
-        if os.path.isdir(path / f):
-            c = file_listing(path / f, str(f))["children"]
-            children += [{"title": f, "children": c}]
-        else:
-            md5 = hashlib.md5(open(path / f, "rb").read()).hexdigest()
-            size = os.path.getsize(path / f)
-            children += [{"title": f + f" (size: {size}; md5sum: {md5})", "size": size, "md5sum": md5}]
+    if path and Path(path).exists() and Path(path).is_dir():
+        for f in os.listdir(path):
+            if len(children) > 5:
+                children += [{"title": "..."}]
+                break
+
+            if os.path.isdir(path / f):
+                c = file_listing(path / f, str(f))["children"]
+                children += [{"title": f, "children": c}]
+            else:
+                md5 = hashlib.md5(open(path / f, "rb").read()).hexdigest()
+                size = os.path.getsize(path / f)
+                children += [{"title": f + f" (size: {size}; md5sum: {md5})", "size": size, "md5sum": md5}]
 
     current_item = {"title": title}
     if len(children) > 0:

frontend/src/Datasets.vue

@@ -84,6 +84,9 @@
         let ret : Record<string, string> = {}
         for (let resource_type of Object.keys(mirrors)) {
           for (let resource_name of Object.keys(mirrors[resource_type])) {
+            if (resource_type.includes('md5_sum') || resource_type.includes('subdirectory') || resource_type.includes('rename_to')) {
+              continue
+            }
             ret[resource_name + ' (' + resource_type + ')'] = mirrors[resource_type][resource_name]
           }
         }

frontend/src/components/EditDataset.vue

@@ -194,6 +194,9 @@
       },
       watch: {
         evaluation_type(new_value, old_value) {
+          if (this.dataset_id_from_props) {
+            return
+          }
           if (new_value != old_value && new_value == 'eval-1') {
             this.git_runner_image = "ubuntu:18.04"
             this.git_runner_command = "echo 'this is no real evaluator'"