remove assert; it is a false positive (#2741)

pola-rs · Feb 23, 2022 · c101f83 · c101f83
1 parent aa249e4
commit c101f83
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/polars/polars-lazy/src/physical_plan/expressions/mod.rs b/polars/polars-lazy/src/physical_plan/expressions/mod.rs
@@ -361,10 +361,12 @@ impl<'a> AggregationContext<'a> {
                 // because this is lazy, we first must to update the groups
                 // by calling .groups()
                 self.groups();
-                assert!(
-                    self.groups.len() <= s.len(),
-                    "implementation error groups are out of bounds; please open an issue"
-                );
+                #[cfg(debug_assertions)]
+                {
+                    if self.groups.len() > s.len() {
+                        eprintln!("groups may be out of bounds; more groups than elements in a series is only possible in dynamic groupby")
+                    }
+                }
 
                 let out = s
                     .agg_list(&self.groups)