From a31c40a1a05af0faeff25655fc1735a14da19ee9 Mon Sep 17 00:00:00 2001 From: Emilia Date: Sat, 13 May 2023 11:45:45 -0400 Subject: [PATCH] Removed explicit pooling transfer The handling of destination interleaves is robust enough that this behaviour comes for free---no need to duplicate. --- src/data/transfer_region.rs | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/src/data/transfer_region.rs b/src/data/transfer_region.rs index ccd806e..69ab15f 100644 --- a/src/data/transfer_region.rs +++ b/src/data/transfer_region.rs @@ -69,8 +69,6 @@ impl TransferRegion<'_> { let il_dest = self.interleave_dest.unwrap_or((1,1)); let il_source = self.interleave_source.unwrap_or((1,1)); - if il_source.0 == 0 { let il_source = (1,il_source.1); } - if il_source.1 == 0 { let il_source = (il_source.1,il_source.0); } let source_corners: ((u8,u8),(u8,u8)) = self.source_region.try_into() .expect("Source region should not be a point"); @@ -80,22 +78,6 @@ impl TransferRegion<'_> { // If the transfer is not a pooling transfer (interleave == 0) // then we *will* have injectivity. - // First we'll handle ij-pooling transfers, since they're the simplest. - // NOTE: I'm going to say that these transfers always go to the primary corner, - // because I can't envision how an ij-pooling replicate transfer would work. - if il_dest == (0,0) { - return Box::new(move |(i,j)| { // All we're moving here is source_wells - if source_wells.contains(&(i,j)) { - match self.dest_region { - Region::Point((i0, j0)) => Some((i0,j0)), - Region::Rect((i0,j0),_) => Some((i0,j0)) - } - } else { - None - } - }) - } - // Non-replicate transfers: if let Region::Point((x,y)) = self.dest_region { return Box::new(move |(i,j)| {