create_many_to_one_relationship() is now just create_relationship()

Author: apdavison

examples/generated_data.py

@@ -56,7 +56,7 @@ def generate_block(n_segments=3, n_channels=4, n_units=3,
                 seg.spiketrains.append(train)
                 u.spiketrains.append(train)
 
-    block.create_many_to_one_relationship()
+    block.create_relationship()
     return block
 
 

neo/core/container.py

@@ -166,12 +166,9 @@ class Container(BaseNeo):
                                          object recursively that are of a
                                          particular class.
 
-        :create_many_to_one_relationship(**args): For each child of the current
-                                                  object that can only have a
-                                                  single parent, set its parent
-                                                  to be the current object.
-
-        :create_relationship(**args): Same as :create_many_to_one_relationship:
+        :create_relationship(**args): For each child of the current
+                                      object, set its parent
+                                      to be the current object.
 
         :merge(**args): Annotations are merged based on the rules of
                         :merge_annotations:.  Child objects with the same name
@@ -209,7 +206,7 @@ def __init__(self, name=None, description=None, file_origin=None,
                          file_origin=file_origin, **annotations)
 
     @property
-    def _single_child_objects(self):
+    def _child_objects(self):
         """
         Child objects that have a single parent.
         """
@@ -233,34 +230,20 @@ def _data_child_containers(self):
                       self._data_child_objects])
 
     @property
-    def _single_child_containers(self):
+    def _child_containers(self):
         """
         Containers for child objects with a single parent.
         """
         return tuple([_container_name(child) for child in
-                      self._single_child_objects])
-
-    @property
-    def _child_objects(self):
-        """
-        All types for child objects.
-        """
-        return self._single_child_objects
-
-    @property
-    def _child_containers(self):
-        """
-        All containers for child objects.
-        """
-        return self._single_child_containers
+                      self._child_objects])
 
     @property
     def _single_children(self):
         """
         All child objects that can only have single parents.
         """
         childs = [list(getattr(self, attr)) for attr in
-                  self._single_child_containers]
+                  self._child_containers]
         return tuple(sum(childs, []))
 
     @property
@@ -405,18 +388,11 @@ def list_children_by_class(self, cls):
             objs.extend(getattr(child, container_name, []))
         return objs
 
-    def create_many_to_one_relationship(self, force=False, recursive=True):
+    def create_relationship(self, force=False, recursive=True):
         """
         For each child of the current object that can only have a single
         parent, set its parent to be the current object.
-
-        Usage:
-        >>> a_block.create_many_to_one_relationship()
-        >>> a_block.create_many_to_one_relationship(force=True)
-
-        If the current object is a :class:`Block`, you want to run
-        populate_RecordingChannel first, because this will create new objects
-        that this method will link up.
+        For children of the current object, put the current object in the parent list.
 
         If force is True overwrite any existing relationships
         If recursive is True descend into child objects and create
@@ -429,30 +405,7 @@ def create_many_to_one_relationship(self, force=False, recursive=True):
                 setattr(child, parent_name, self)
         if recursive:
             for child in self.container_children:
-                child.create_many_to_one_relationship(force=force,
-                                                      recursive=True)
-
-    def create_relationship(self, force=False, append=True, recursive=True):
-        """
-        For each child of the current object that can only have a single
-        parent, set its parent to be the current object.
-        For children of the current object that can have more than one parent
-        of this type, put the current object in the parent list.
-
-        If the current object is a :class:`Block`, you want to run
-        populate_RecordingChannel first, because this will create new objects
-        that this method will link up.
-
-        If force is True overwrite any existing relationships
-        If append is True add it to the list, otherwise overwrite the list.
-        If recursive is True descend into child objects and create
-        relationships there
-        """
-        self.create_many_to_one_relationship(force=force, recursive=False)
-        if recursive:
-            for child in self.container_children:
-                child.create_relationship(force=force, append=append,
-                                          recursive=True)
+                child.create_relationship(force=force, recursive=True)
 
     def __deepcopy__(self, memo):
         """

neo/io/asciisignalio.py

@@ -292,7 +292,7 @@ def read_segment(self, lazy=False):
                                            name='Column %d' % i)
                     seg.analogsignals.append(ana_sig)
 
-        seg.create_many_to_one_relationship()
+        seg.create_relationship()
         return seg
 
     def read_metadata(self):

neo/io/asciispiketrainio.py

@@ -107,7 +107,7 @@ def read_segment(self,
                 sptr.annotate(channel_index=i)
                 seg.spiketrains.append(sptr)
 
-        seg.create_many_to_one_relationship()
+        seg.create_relationship()
         return seg
 
     def write_segment(self, segment,

neo/io/basefromrawio.py

@@ -178,7 +178,7 @@ def read_block(self, block_index=0, lazy=False,
                 for c, sptr in enumerate(seg.spiketrains):
                     st_groups[c].add(sptr)
 
-        bl.create_many_to_one_relationship()
+        bl.create_relationship()
 
         return bl
 
@@ -278,7 +278,7 @@ def read_segment(self, block_index=0, seg_index=0, lazy=False,
                 e.segment = seg
                 seg.epochs.append(e)
 
-        seg.create_many_to_one_relationship()
+        seg.create_relationship()
         return seg
 
     def get_sub_signal_streams(self, signal_group_mode='group-by-same-units'):

neo/io/baseio.py

@@ -126,7 +126,7 @@ def read(self, lazy=False, **kargs):
             bl = Block(name='One segment only')
             seg = self.read_segment(lazy=lazy, **kargs)
             bl.segments.append(seg)
-            bl.create_many_to_one_relationship()
+            bl.create_relationship()
             return [bl]
         else:
             raise NotImplementedError

neo/io/brainwaredamio.py

@@ -129,7 +129,7 @@ def read_block(self, lazy=False, **kargs):
 
         # create the objects to store other objects
         gr = Group(file_origin=self._filename)
-        
+
         # load objects into their containers
         block.groups.append(gr)
 
@@ -149,7 +149,7 @@ def read_block(self, lazy=False, **kargs):
         # remove the file object
         self._fsrc = None
 
-        block.create_many_to_one_relationship()
+        block.create_relationship()
         return block
 
     # -------------------------------------------------------------------------

neo/io/brainwaref32io.py

@@ -158,7 +158,7 @@ def read_block(self, lazy=False, **kargs):
             while res:
                 res = self.__read_id()
 
-        block.create_many_to_one_relationship()
+        block.create_relationship()
 
         # cleanup attributes
         self._fsrc = None

neo/io/brainwaresrcio.py

@@ -279,7 +279,7 @@ def read_next_block(self, **kargs):
                 raise
 
         # since we read at a Block level we always do this
-        self._blk.create_many_to_one_relationship()
+        self._blk.create_relationship()
 
         # put the Block in a local object so it can be gargabe collected
         blockobj = self._blk

neo/io/klustakwikio.py

@@ -181,7 +181,7 @@ def read_block(self, lazy=False):
                 u.add(st)
                 seg.spiketrains.append(st)
 
-        block.create_many_to_one_relationship()
+        block.create_relationship()
         return block
 
     # Helper hidden functions for reading