Use Real instead of float for numbers, to avoid integers triggering validation errors

Author: apdavison

pipeline/src/properties.py

@@ -6,6 +6,7 @@
 
 from datetime import datetime, date
 from collections import defaultdict
+from numbers import Real
 from typing import Optional, Union, Iterable
 
 from .registry import lookup
@@ -172,6 +173,8 @@ def deserialize_item(item):
                 return IRI(item)
             elif float in self.types:
                 return item
+            elif Real in self.types:
+                return item
             elif int in self.types:
                 return item
             elif datetime in self.types:

pipeline/tests/utils.py

@@ -2,6 +2,7 @@
 Utility functions and constants for openMINDS tests
 """
 
+from numbers import Real
 import random
 import string
 from datetime import date, datetime
@@ -34,7 +35,7 @@ def build_fake_node(cls, depth=0):
                 # todo: check for property.formatting
                 value = "".join(random.choices(string.printable, k=random.randint(0, 40)))
                 data[property.name] = value
-            elif float in property.types:
+            elif Real in property.types:
                 value = 100 * random.random()
                 data[property.name] = value
             elif int in property.types:

pipeline/translator.py

@@ -1,5 +1,6 @@
 from datetime import date, time, datetime
 import json
+from numbers import Real
 import os.path
 import re
 from typing import List, Optional, Dict
@@ -87,7 +88,7 @@ def get_type(property):
             type_map = {
                 "string": "str",
                 "integer": "int",
-                "number": "float",
+                "number": "Real",
                 "date": "date",
                 "date-time": "datetime",
                 "time": "time",
@@ -209,6 +210,7 @@ def filter_instance(instance):
             "time": "from datetime import time",
             "IRI": "from openminds.base import IRI",
             "[datetime, time]": "from datetime import datetime, time",
+            "Real": "from numbers import Real"
         }
         extra_imports = set()
         for property in self.context["properties"]: