Omit whole pattern targets from branch subpipelines when possible

NEWS.md

@@ -15,6 +15,7 @@
 * Reduce memory consumption by storing buds and branches as lightweight references when `memory` is `"transient"` (#1364).
 * Replace the `memory` class with the new `lookup` class.
 * Implement `memory = "auto"` to select transient memory for dynamic branches and persistent memory for other targets (#1371).
+* Omit whole pattern targets from branch subpipelines when possible. Should reduce memory consumption in some cases.
 
 # targets 1.8.0
 

R/class_target.R

@@ -159,10 +159,13 @@ target_deps_deep <- function(target, pipeline) {
       target_get_children(pipeline_get_target(pipeline, dep))
     })
   )
-  parents <- map_chr(deps, function(dep) {
-    target_get_parent(pipeline_get_target(pipeline, dep))
-  })
-  unique(c(deps, children, parents))
+  patterns <- unlist(
+    map(deps, function(dep) {
+      target <- pipeline_get_target(pipeline, dep)
+      if_any(inherits(target, "tar_bud"), target_get_parent(target), NULL)
+    })
+  )
+  unique(c(deps, children, patterns))
 }
 
 target_downstream_branching <- function(target, pipeline, scheduler) {