Merge pull request #57 from eulerlab/bugfix/tooLongKey

Set length of primary keys of type varchar to 32
eulerlab · Mar 18, 2024 · 085d017 · 085d017
2 parents fe06075 + 25a6d1e
commit 085d017
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 8 deletions.
diff --git a/djimaging/tables/core/field.py b/djimaging/tables/core/field.py
@@ -21,7 +21,7 @@ def definition(self):
         definition = """
         # Recording fields
         -> self.experiment_table
-        field   :varchar(191)          # string identifying files corresponding to field
+        field   :varchar(32)          # string identifying files corresponding to field
         ---
         fromfile: varchar(191)  # info extracted from which file?
         absx: float  # absolute position of the center (of the cropped field) in the x axis as recorded by ScanM
@@ -196,7 +196,7 @@ def definition(self):
             definition = """
                 # Stack median (over time of the available channels)
                 -> master
-                ch_name : varchar(191)  # name of the channel
+                ch_name : varchar(32)  # name of the channel
                 ---
                 ch_average :longblob  # Stack median over time
                 """

diff --git a/djimaging/tables/core/presentation.py b/djimaging/tables/core/presentation.py
@@ -25,7 +25,7 @@ def definition(self):
         -> self.field_table
         -> self.stimulus_table
         -> self.params_table
-        condition             :varchar(191)     # condition (pharmacological or other)
+        condition             :varchar(32)     # condition (pharmacological or other)
         ---
         {self.filepath}     :varchar(191)     # path to h5 file
         trigger_flag          :tinyint unsigned # Are triggers as expected (1) or not (0)?
@@ -80,7 +80,7 @@ def definition(self):
             definition = """
             # Stack median (over time of the available channels)
             -> master
-            ch_name : varchar(191)  # name of the channel
+            ch_name : varchar(32)  # name of the channel
             ---
             ch_average : longblob  # Stack median over time
             """

diff --git a/djimaging/tables/core/stimulus.py b/djimaging/tables/core/stimulus.py
@@ -48,7 +48,7 @@ class StimulusTemplate(dj.Manual):
     def definition(self):
         definition = """
         # Light stimuli
-        stim_name           :varchar(191)       # Unique string identifier
+        stim_name           :varchar(32)       # Unique string identifier
         ---
         alias               :varchar(9999)      # Strings (_ seperator) to identify this stimulus, not case sensitive!
         stim_family=""      :varchar(191)       # To group stimuli (e.g. gChirp and lChirp) for downstream processing 

diff --git a/djimaging/tables/core/userinfo.py b/djimaging/tables/core/userinfo.py
@@ -14,7 +14,7 @@ def definition(self):
         definition = """
         # Info for decoding file names
     
-        experimenter                    :varchar(191)  # name of the experimenter
+        experimenter                    :varchar(32)  # name of the experimenter
         ---     
         data_dir                        :varchar(191)  # path to header file, used for computed tables
         datatype_loc                    :tinyint       # string location for datatype (e.g. SMP)

diff --git a/djimaging/tables/core_autorois/field.py b/djimaging/tables/core_autorois/field.py
@@ -24,7 +24,7 @@ def definition(self):
         # Recording fields
         -> self.experiment_table
         -> self.raw_params_table
-        field   :varchar(191)          # string identifying files corresponding to field
+        field   :varchar(32)          # string identifying files corresponding to field
         """
 
         if self._include_conditions:

diff --git a/djimaging/tables/core_autorois/presentation.py b/djimaging/tables/core_autorois/presentation.py
@@ -25,7 +25,7 @@ def definition(self):
         -> self.field_table
         -> self.stimulus_table
         -> self.raw_params_table
-        condition             :varchar(63)     # condition (pharmacological or other)
+        condition             :varchar(32)     # condition (pharmacological or other)
         ---
         {self.filepath}       :varchar(191)     # path to h5 file
         trigger_flag          :tinyint unsigned # Are triggers as expected (1) or not (0)?