New: Return default value of fieldvalue when possible

eppy/EPlusInterfaceFunctions/parse_idd.py

@@ -140,8 +140,6 @@ def embedgroupdata(extract_func, fname, debug):
     return blocklst, commlst, commdct
 
 
-@make_idd_index
-@embedgroupdata
 def extractidddata(fname, debug=False):
     """
     extracts all the needed information out of the idd file
@@ -375,7 +373,18 @@ def extractidddata(fname, debug=False):
     # commlst = group2commlst(commlst, glist)
     # commdct = group2commdct(commdct, glist)
 
-    return blocklst, commlst, commdct
+    # add group information to commlst and commdct
+    glist = iddgroups.iddtxt2grouplist(astr)
+    commlst = iddgroups.group2commlst(commlst, glist)
+    commdct = iddgroups.group2commdct(commdct, glist)
+
+    # Make IDD Index
+    name2refs = iddindex.makename2refdct(commdct)
+    ref2namesdct = iddindex.makeref2namesdct(name2refs)
+    idd_index = dict(name2refs=name2refs, ref2names=ref2namesdct)
+    commdct = iddindex.ref2names2commdct(ref2namesdct, commdct)
+
+    return blocklst, commlst, commdct, idd_index
     # give blocklst a better name :-(
 
 

eppy/EPlusInterfaceFunctions/readidf.py

@@ -25,7 +25,7 @@ def readidf(idfname):
     iddfile = "Energy+.idd"
     # iddfile = './EPlusInterfaceFunctions/E1.idd' # TODO : can the path name be not hard coded
     iddtxt = open(iddfile, "r").read()
-    block, commlst, commdct = parse_idd.extractidddata(iddfile)
+    block, commlst, commdct, idd_index = parse_idd.extractidddata(iddfile)
 
     theidd = eplusdata.Idd(block, 2)
     data = eplusdata.Eplusdata(theidd, idfname)
@@ -38,7 +38,7 @@ def readiddidf(idfname):
     iddfile = "Energy+.idd"
     # iddfile = './EPlusInterfaceFunctions/E1.idd' # TODO : can the path name be not hard coded
     iddtxt = open(iddfile, "r").read()
-    block, commlst, commdct = parse_idd.extractidddata(iddfile)
+    block, commlst, commdct, idd_index = parse_idd.extractidddata(iddfile)
 
     theidd = eplusdata.Idd(block, 2)
     data = eplusdata.Eplusdata(theidd, idfname)
@@ -51,7 +51,7 @@ def readiddstuff(idfname):
     iddfile = "Energy+.idd"
     # iddfile = './EPlusInterfaceFunctions/E1.idd' # TODO : can the path name be not hard coded
     iddtxt = open(iddfile, "r").read()
-    block, commlst, commdct = parse_idd.extractidddata(iddfile)
+    block, commlst, commdct, idd_index = parse_idd.extractidddata(iddfile)
 
     theidd = eplusdata.Idd(block, 2)
     data = eplusdata.Eplusdata(theidd, idfname)
@@ -64,7 +64,7 @@ def readdatacommlst(idfname):
     iddfile = "Energy+.idd"
     # iddfile = './EPlusInterfaceFunctions/E1.idd' # TODO : can the path name be not hard coded
     iddtxt = open(iddfile, "r").read()
-    block, commlst, commdct = parse_idd.extractidddata(iddfile)
+    block, commlst, commdct, idd_index = parse_idd.extractidddata(iddfile)
 
     theidd = eplusdata.Idd(block, 2)
     data = eplusdata.Eplusdata(theidd, idfname)

eppy/bunch_subclass.py

@@ -317,13 +317,26 @@ def __getattr__(self, name):
         elif name in self.fieldnames:
             i = self.fieldnames.index(name)
             try:
-                return self.fieldvalues[i]
+                i_ = self.fieldvalues[i]
+                if i_ == "":
+                    return self.get_default(name)
+                else:
+                    return i_
             except IndexError:
-                return ""
+                # try to return the default value if defined in IDD.
+                return self.get_default(name)
         else:
             astr = "unable to find field %s" % (name,)
             raise BadEPFieldError(astr)
 
+    def get_default(self, name):
+        if "default" in self.getfieldidd(name).keys():
+            _type = _parse_idd_type(self, name)
+            default_ = next(iter(self.getfieldidd_item(name, "default")), None)
+            return _type(default_)
+        else:
+            return ""
+
     def __getitem__(self, key):
         if key in ("obj", "objls", "objidd", "__functions", "__aliases", "theidf"):
             return super(EpBunch, self).__getitem__(key)
@@ -395,6 +408,30 @@ def __dir__(self):
         return super(EpBunch, self).__dir__() + fnames + func_names
 
 
+def _parse_idd_type(epbunch, name):
+    """parse the fieldvalue type into a python type. eg.: 'real' returns
+    'float'.
+
+    Possible types are:
+        - integer -> int
+        - real -> float
+        - alpha -> str          (arbitrary string),
+        - choice -> str         (alpha with specific list of choices, see \key)
+        - object-list -> str    (link to a list of objects defined elsewhere, see \object-list and \reference)
+        - external-list -> str  (uses a special list from an external source, see \external-list)
+        - node -> str           (name used in connecting HVAC components)
+    """
+    _type = next(iter(epbunch.getfieldidd_item(name, "type")), "").lower()
+    if _type == "real":
+        return float
+    elif _type == "alpha":
+        return str
+    elif _type == "integer":
+        return int
+    else:
+        return str
+
+
 def getrange(bch, fieldname):
     """get the ranges for this field"""
     keys = ["maximum", "minimum", "maximum<", "minimum>", "type"]

eppy/bunchhelpers.py

@@ -11,13 +11,13 @@
 from __future__ import print_function
 from __future__ import unicode_literals
 
-from string import ascii_letters, digits
+import re
+pattern = re.compile(r"[^a-zA-Z0-9\s]+")
 
 
-def onlylegalchar(name):
+def onlylegalchar(name: str):
     """return only legal chars"""
-    legalchar = ascii_letters + digits + " "
-    return "".join([s for s in name[:] if s in legalchar])
+    return pattern.sub("", name)
 
 
 def makefieldname(namefromidd):

eppy/runner/run_functions.py

@@ -12,6 +12,7 @@
 from __future__ import print_function
 from __future__ import unicode_literals
 
+import glob
 import os
 import platform
 import pydoc

tests/test_modeleditor.py

@@ -328,7 +328,8 @@ def test_newidfobject():
     sim_deftrue = idf.newidfobject("SimulationControl".upper(), defaultvalues=True)
     assert sim_deftrue.Do_Zone_Sizing_Calculation == "No"
     sim_deffalse = idf.newidfobject("SimulationControl".upper(), defaultvalues=False)
-    assert sim_deffalse.Do_Zone_Sizing_Calculation == ""
+    assert sim_deffalse.Do_Zone_Sizing_Calculation == "No"
+    assert str(sim_deffalse) == "\n    SIMULATIONCONTROL,;\n"
 
 
 def test_newidfobject_warning():