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

@@ -422,6 +422,8 @@ class Dataset:
         :rtype: list
         """
         return [
+            # TODO see commit f5208db0 - this will do unexpected things if
+            #      self.dateOfCollection is a naive datetime object (think DST transitions...)
             self.dateOfCollection + datetime.timedelta(seconds=x)
             for x in self.independentVariable
         ]
@@ -894,6 +896,9 @@ class Dataset:
         self.PIAffiliation = "Musterinstitut"
         self.dataSourceDescription = "Musterdatenprodukt"
         self.missionName = "MUSTEREX"
+
+        # TODO: not 100%  sure if this is relevant here, but those dates should refer to UTC
+        # - if you leave the datetime objects naive (no tzinfo set), they represent local time.
         self.dateOfCollection = datetime.datetime.today()
         self.dateOfRevision = datetime.datetime.today()
         self.dataIntervalCode = [0.0]
@@ -921,6 +926,8 @@ class Dataset:
             else:
                 self.inputFhandle = f
 
+        # TODO should we add a filename characters check, as we have it for the variable names?
+
             self.readHeader(splitChar)
             if loadData:
                 self.endDefineMode()