Remove AutoPopulate.target property

The new implementation always populates self - the target property is no
longer needed. All references to self.target replaced with self.

Author: claude

src/datajoint/autopopulate.py

@@ -74,7 +74,7 @@ def _rename_attributes(table, props):
             )
 
         if self._key_source is None:
-            parents = self.target.parents(primary=True, as_objects=True, foreign_key_info=True)
+            parents = self.parents(primary=True, as_objects=True, foreign_key_info=True)
             if not parents:
                 raise DataJointError("A table must have dependencies from its primary key for auto-populate to work")
             self._key_source = _rename_attributes(*parents[0])
@@ -151,15 +151,6 @@ def make(self, key):
         self.make_insert(key, *computed_result)
         yield
 
-    @property
-    def target(self):
-        """
-        :return: table to be populated.
-        In the typical case, dj.AutoPopulate is mixed into a dj.Table class by
-        inheritance and the target is self.
-        """
-        return self
-
     @property
     def jobs(self):
         """
@@ -173,7 +164,7 @@ def jobs(self):
         if self._jobs_table is None:
             from .jobs import JobsTable
 
-            self._jobs_table = JobsTable(self.target)
+            self._jobs_table = JobsTable(self)
         return self._jobs_table
 
     def _jobs_to_do(self, restrictions):
@@ -198,7 +189,7 @@ def _jobs_to_do(self, restrictions):
             raise DataJointError(
                 "The populate target lacks attribute %s "
                 "from the primary key of key_source"
-                % next(name for name in todo.heading.primary_key if name not in self.target.heading)
+                % next(name for name in todo.heading.primary_key if name not in self.heading)
             )
         except StopIteration:
             pass
@@ -281,7 +272,7 @@ def handler(signum, frame):
         else:
             # Legacy behavior: get keys from key_source
             if keys is None:
-                keys = (self._jobs_to_do(restrictions) - self.target).fetch("KEY", limit=limit)
+                keys = (self._jobs_to_do(restrictions) - self).fetch("KEY", limit=limit)
 
         if order == "reverse":
             keys.reverse()
@@ -390,15 +381,15 @@ def _populate1(self, key, jobs, suppress_errors, return_exception_objects, make_
         if not is_generator:
             self.connection.start_transaction()
 
-        if key in self.target:  # already populated
+        if key in self:  # already populated
             if not is_generator:
                 self.connection.cancel_transaction()
             if jobs is not None:
                 # Job already done - mark complete or delete
                 jobs.complete(key, duration=0)
             return False
 
-        logger.debug(f"Making {key} -> {self.target.full_table_name}")
+        logger.debug(f"Making {key} -> {self.full_table_name}")
         self.__class__._allow_insert = True
 
         try:
@@ -429,7 +420,7 @@ def _populate1(self, key, jobs, suppress_errors, return_exception_objects, make_
                 exception=error.__class__.__name__,
                 msg=": " + str(error) if str(error) else "",
             )
-            logger.debug(f"Error making {key} -> {self.target.full_table_name} - {error_message}")
+            logger.debug(f"Error making {key} -> {self.full_table_name} - {error_message}")
 
             # Only log errors from inside make() - not collision errors
             if jobs is not None:
@@ -456,7 +447,7 @@ def _populate1(self, key, jobs, suppress_errors, return_exception_objects, make_
         else:
             self.connection.commit_transaction()
             duration = time.time() - start_time
-            logger.debug(f"Success making {key} -> {self.target.full_table_name}")
+            logger.debug(f"Success making {key} -> {self.full_table_name}")
             if jobs is not None:
                 jobs.complete(key, duration=duration)
             return True
@@ -470,7 +461,7 @@ def progress(self, *restrictions, display=False):
         """
         todo = self._jobs_to_do(restrictions)
         total = len(todo)
-        remaining = len(todo - self.target)
+        remaining = len(todo - self)
         if display:
             logger.info(
                 "%-20s" % self.__class__.__name__