code refactor

related to #8 (closed):

• I kept changes per commit small, so the number of commits got kind of numerous...

src/icartt/dataset.py

@@ -125,7 +125,7 @@ class DataStore2110(collections.UserDict):
         self.add(**newdata)
 
     def addBulkDep(self, ivar, raw):
-        nlines, nvars = raw.shape
+        nlines, nvars = raw.shape # nvars not used
         self._addDeplines(ivar, raw, nlines)
 
     def _addDeplines(self, ivar, raw, n):
@@ -491,6 +491,10 @@ class Dataset:
             self.format = Formats(int(dmp[1]))
         except:
             raise ValueError("ICARTT format {:d} not implemented".format(dmp[1]))
+            # TODO except clause could be re-written like
+            # except ValueError as ve:
+            #     raise ValueError(f"ICARTT format {dmp[1]:d} not implemented") from ve
+            # to show the error trace
 
         if len(dmp) > 2:
             self.version = dmp[2]
@@ -691,7 +695,7 @@ class Dataset:
     def read(self, splitChar=","):
         """Read ICARTT data and header"""
         self.readHeader(splitChar)
-        self.endDefineMode(splitChar)
+        self.endDefineMode(splitChar) # TODO: endDefineMode does not take arg splitChar
         self.readData(splitChar)
 
     def makeFileName(self, dateFormat="%Y%m%d"):