rustyml 0.17.0

//! 1D depthwise separable convolution layer
//!
//! [`SeparableConv1D`] runs a depthwise stage that convolves each input channel on its own,
//! followed by a pointwise stage that mixes the depthwise outputs with a 1-tap convolution.
//! This factorization can use fewer parameters and less work than a standard convolution with
//! the same channel counts. `SeparableConv1DCache` holds the tensors the layer parks between
//! its forward and backward passes.

use crate::error::Error;
use crate::neural_network::layers::ParamCounts;
use crate::neural_network::layers::activation::{Activation, ActivationCache};
use crate::neural_network::layers::conv_op_helpers::{
    DepthwiseGeometry, depthwise_backward, depthwise_forward,
};
use crate::neural_network::layers::convolution::PaddingType;
use crate::neural_network::layers::convolution::convolution_engine::{
    ConvPadding, conv_backward, conv_forward, effective_kernel,
};
use crate::neural_network::layers::convolution::validation::{
    valid_output_size, validate_depth_multiplier, validate_dilation, validate_filters,
    validate_input_shape_1d, validate_kernel_size_1d, validate_strides_1d,
    validate_valid_kernel_fits,
};
use crate::neural_network::layers::validation::{
    start_build, validate_built_input, validate_optional_weight, validate_weight_shape,
};
use crate::neural_network::layers::{built_layer_shape_functions, named_weight_layer_functions};
use crate::neural_network::traits::{LayerBase, ParamRef, UnaryLayer};
use crate::neural_network::{Ctx, Fans, Initializer, Shape, Tensor};
use ndarray::{Array1, Array3};

/// A 1D separable convolutional layer
///
/// Runs a depthwise step followed by a pointwise step. This can use fewer parameters and less
/// work than a standard convolution. Input shape is \[batch_size, length, channels\].
/// Intermediate depthwise output shape is \[batch_size, length', channels * depth_multiplier\].
/// Final output shape is \[batch_size, length', filters\]
///
/// The intermediate channel for input channel `c` and multiplier index `m` is
/// `c * depth_multiplier + m`. It is also the row order of the
/// pointwise weight `\[1, channels * depth_multiplier, filters\]`. This aligns the 2 stages
/// with no repacking between them
///
/// The separable convolution runs 2 steps:
/// 1. Depthwise convolution: convolves each input channel with its own set of filters
/// 2. Pointwise convolution: a 1-tap convolution that combines the depthwise outputs
///
/// This is the 1D form of
/// [`SeparableConv2D`](crate::neural_network::layers::convolution::SeparableConv2D)
///
/// # Examples
///
/// ```rust
/// use rustyml::neural_network::Shape;
/// use rustyml::neural_network::sequential::SequentialBuilder;
/// use rustyml::neural_network::layers::*;
/// use rustyml::neural_network::optimizers::*;
/// use rustyml::neural_network::losses::*;
/// use ndarray::Array3;
///
/// // input tensor: [batch_size, length, channels].
/// let x = Array3::ones((2, 32, 3)).into_dyn();
///
/// // target tensor.
/// let y = Array3::ones((2, 32, 16)).into_dyn();
///
/// let mut model = SequentialBuilder::new()
///     .add(SeparableConv1D::new(
///         16,                          // filters
///         3,                           // kernel_size
///         1,                           // stride
///         1,                           // depth_multiplier
///         Activation::ReLU,            // activation
///     ).unwrap().with_padding(PaddingType::Same))
///     .build(&Shape::known(x.shape()))
///     .unwrap();
/// model.compile(RMSprop::new(0.001, 0.9, 1e-8, 0.0).unwrap(), MeanSquaredError::new());
///
/// model.summary();
/// model.fit(&x, &y, 3).unwrap();
/// ```
#[derive(Debug)]
pub struct SeparableConv1D {
    /// Number of output channels from the pointwise convolution
    filters: usize,
    /// Number of input channels, which
    /// [`UnaryLayer::build`](crate::neural_network::traits::UnaryLayer::build) reads from the input
    /// shape
    channels: usize,
    /// Depthwise convolution kernel size along the length axis
    kernel_size: usize,
    /// Stride of the convolution along the length axis
    stride: usize,
    /// Tap spacing of the depthwise kernel. 1 gives a solid kernel
    dilation_rate: usize,
    /// Padding applied to the length axis (`Valid` or `Same`)
    padding: PaddingType,
    /// Number of depthwise filters per input channel
    depth_multiplier: usize,
    /// Depthwise filters with shape \[kernel_size, channels, depth_multiplier\]
    depthwise_weights: Array3<f32>,
    /// Pointwise filters with shape \[1, channels * depth_multiplier, filters\]
    pointwise_weights: Array3<f32>,
    /// Bias vector with shape \[filters\]
    ///
    /// The array stays allocated when `use_bias` is false. The forward pass and weight rosters
    /// exclude it.
    /// The pointwise stage adds nothing, `weights` hides the array, and `parameters_mut` never
    /// yields it, so a bias-free layer holds it and no more
    bias: Array1<f32>,
    /// Activation applied to the layer output
    activation: Activation,
    /// Shape the layer was built for, batch axis first. `None` before the build
    built: Option<Shape>,
    /// Seed of the weight draw, or `None` to take the global seed or entropy
    random_state: Option<u64>,
    /// Whether the layer adds a bias to the pointwise output
    use_bias: bool,
}

impl SeparableConv1D {
    /// Creates a new 1D separable convolutional layer
    ///
    /// The build draws weights with Xavier (Glorot) uniform initialization and sets biases to 0.
    ///
    /// # Parameters
    ///
    /// - `filters` - Number of output channels from the pointwise convolution
    /// - `kernel_size` - Size of the depthwise convolution kernel along the length axis
    /// - `stride` - Stride of the convolution along the length axis
    /// - `depth_multiplier` - Number of depthwise convolution filters per input channel
    /// - `activation` - Activation applied to the output
    ///
    /// # Returns
    ///
    /// - `Result<Self, Error>` - A new `SeparableConv1D` layer instance or an error
    ///
    /// # Notes
    ///
    /// Padding defaults to
    /// [`PaddingType::Valid`]. Choose
    /// [`PaddingType::Same`] with
    /// [`SeparableConv1D::with_padding`]. The depthwise kernel is solid by default. Space its taps
    /// out with [`SeparableConv1D::with_dilation_rate`].
    ///
    /// The layer seeds weights from the global seed or entropy by default. For reproducible
    /// initialization, set a seed with [`SeparableConv1D::with_random_state`]
    ///
    /// # Errors
    ///
    /// - `Error::InvalidParameter` - If `filters` is 0
    /// - `Error::InvalidParameter` - If `kernel_size` or `stride` is 0
    /// - `Error::InvalidParameter` - If `depth_multiplier` is 0
    /// - `Error::InvalidParameter` - If the activation carries an unusable parameter (see
    ///   [`Activation::validate`])
    pub fn new(
        filters: usize,
        kernel_size: usize,
        stride: usize,
        depth_multiplier: usize,
        activation: impl Into<Activation>,
    ) -> Result<Self, Error> {
        validate_filters(filters)?;
        validate_kernel_size_1d(kernel_size)?;
        validate_strides_1d(stride)?;
        validate_depth_multiplier(depth_multiplier)?;
        let activation = activation.into();
        activation.validate()?;

        Ok(SeparableConv1D {
            filters,
            channels: 0,
            kernel_size,
            stride,
            dilation_rate: 1,
            padding: PaddingType::Valid,
            depth_multiplier,
            depthwise_weights: Array3::zeros((0, 0, 0)),
            pointwise_weights: Array3::zeros((0, 0, 0)),
            bias: Array1::zeros(0),
            activation,
            built: None,
            random_state: None,
            use_bias: true,
        })
    }

    /// Sets the padding mode (defaults to
    /// [`PaddingType::Valid`])
    ///
    /// # Parameters
    ///
    /// - `padding` - Type of padding to apply (`Valid` or `Same`)
    ///
    /// # Returns
    ///
    /// - `Self` - The updated layer
    pub fn with_padding(mut self, padding: PaddingType) -> Self {
        self.padding = padding;
        self
    }

    /// Sets the tap spacing of the depthwise kernel (defaults to 1)
    ///
    /// A dilation of `d` spaces the depthwise taps `d` cells apart, so `kernel_size` taps span
    /// `(kernel_size - 1) * d + 1` input cells. The window still advances by the stride. The
    /// pointwise stage reads 1 tap, so no dilation can reach it. A dilation of 1 gives a solid
    /// kernel, the same as a convolution with no dilation
    ///
    /// # Parameters
    ///
    /// - `dilation_rate` - Tap spacing along the length axis
    ///
    /// # Returns
    ///
    /// - `Result<Self, Error>` - The updated layer, or an error
    ///
    /// # Notes
    ///
    /// A separable convolution takes a stride above 1 and a dilation above 1 together. Only the
    /// plain and the transposed convolutions reject that pair
    ///
    /// The effective kernel is not bounded by the input length here. Only
    /// [`PaddingType::Valid`] needs it to
    /// fit, and the build applies that rule
    ///
    /// # Errors
    ///
    /// - `Error::InvalidParameter` - If `dilation_rate` is 0
    pub fn with_dilation_rate(mut self, dilation_rate: usize) -> Result<Self, Error> {
        validate_dilation(&[dilation_rate])?;
        self.dilation_rate = dilation_rate;
        Ok(self)
    }

    /// Sets the seed for the depthwise and pointwise weights, and re-initializes them
    /// deterministically
    ///
    /// By default, the layer seeds weights from the global seed or entropy (see
    /// [`crate::random`]). For a built layer, this repeats Xavier/Glorot uniform initialization with `random_state`.
    /// Call it before you assign custom weights or start training. A built layer also resets
    /// its bias to 0
    ///
    /// # Parameters
    ///
    /// - `random_state` - Seed for weight initialization
    ///
    /// # Returns
    ///
    /// - `Self` - The updated layer
    pub fn with_random_state(mut self, random_state: u64) -> Self {
        self.random_state = Some(random_state);
        if self.built.is_some() {
            self.draw_parameters();
        }
        self
    }

    /// Xavier/Glorot uniform initialization of the depthwise and pointwise weight tensors
    ///
    /// Both draws share 1 RNG (threaded depthwise-then-pointwise) so a given seed reproduces the
    /// exact same pair of tensors
    fn draw_parameters(&mut self) {
        let mut rng = crate::random::make_rng(self.random_state);

        // The depth multiplier takes the place of the filter count. See `Fans::conv`
        self.depthwise_weights = Initializer::GlorotUniform.draw(
            (self.kernel_size, self.channels, self.depth_multiplier),
            Fans::conv(self.channels, self.depth_multiplier, self.kernel_size),
            &mut rng,
        );

        // The stored width of the pointwise kernel is already the fan-in, and its 1 tap adds no
        // factor to the fan-out
        let width = self.channels * self.depth_multiplier;
        self.pointwise_weights = Initializer::GlorotUniform.draw(
            (1, width, self.filters),
            Fans::new(width, self.filters),
            &mut rng,
        );

        self.bias = Array1::zeros(self.filters);
    }

    /// Calculates the output length of both stages, which share 1 length
    ///
    /// The pointwise stage has a 1-tap kernel under `Valid` padding, so it leaves the length
    /// unchanged. The depthwise stage alone therefore sets the output length
    ///
    /// # Errors
    ///
    /// - `Error::InvalidInput` - If the padding is `Valid` and the effective kernel is longer than
    ///   the input length
    fn calculate_output_length(&self, input_length: usize) -> Result<usize, Error> {
        match self.padding {
            // The `Valid` rule reads the extent the dilated taps span, not the tap count
            PaddingType::Valid => valid_output_size(
                "SeparableConv1D",
                "length",
                input_length,
                effective_kernel(self.kernel_size, self.dilation_rate),
                self.stride,
            ),
            PaddingType::Same => Ok(input_length.div_ceil(self.stride)),
        }
    }

    /// The depthwise stage's geometry for a given input, as the shared kernel wants it
    ///
    /// The shared kernel names 2 spatial axes, so this fixes the height at 1 and puts the length
    /// on the width axis. A `[batch, length, channels]` tensor and a
    /// `[kernel_size, channels, depth_multiplier]` weight already hold the values in that order,
    /// so neither one needs a copy
    ///
    /// # Errors
    ///
    /// - `Error::InvalidInput` - If the padding is `Valid` and the effective kernel is longer than
    ///   the input length. The build applies the same rule, so a built layer never meets it
    fn depthwise_geometry(&self, input_shape: &[usize]) -> Result<DepthwiseGeometry, Error> {
        let length = input_shape[1];
        let out_length = self.calculate_output_length(length)?;
        let keff = effective_kernel(self.kernel_size, self.dilation_rate);
        let pad = match self.padding {
            PaddingType::Valid => 0,
            PaddingType::Same => ((out_length - 1) * self.stride + keff).saturating_sub(length),
        };
        Ok(DepthwiseGeometry {
            input: (1, length),
            output: (1, out_length),
            channels: input_shape[2],
            depth_multiplier: self.depth_multiplier,
            kernel: (1, self.kernel_size),
            strides: (1, self.stride),
            // The height axis is the placeholder axis, so it stays solid at 1
            dilation: (1, self.dilation_rate),
            pad_before: (0, pad / 2),
        })
    }

    /// Checks a runtime input against the rank, the channel count, and the padding rule the
    /// layer was built for
    ///
    /// The depthwise kernel is sized from the declared channel count, so an input carrying more
    /// channels would read past the end of it. This turns that into an error at the layer
    /// boundary. It also applies the `Valid` fit rule, because a `Valid` window that is longer
    /// than the input length gives no complete window
    fn validate_input(&self, input: &Tensor) -> Result<(), Error> {
        validate_built_input(&self.built, "SeparableConv1D", input.shape())?;
        validate_valid_kernel_fits(
            self.padding.into(),
            &[self.kernel_size],
            &[self.dilation_rate],
            &input.shape()[1..2],
        )?;
        Ok(())
    }

    /// Performs the depthwise convolution stage
    ///
    /// Carries no bias and no activation. Both belong to the pointwise stage that follows, so
    /// this passes `None` for the bias to the shared kernel
    ///
    /// # Errors
    ///
    /// - `Error::InvalidInput` - If the padding is `Valid` and the effective kernel is longer than
    ///   the input length. The build applies the same rule, so a built layer never meets it
    fn depthwise_convolve(&self, input: &Tensor) -> Result<Tensor, Error> {
        let g = self.depthwise_geometry(input.shape())?;
        let batch_size = input.shape()[0];

        let input_std = input.as_standard_layout();
        let src = input_std
            .as_slice()
            .expect("standard-layout array is contiguous");
        let ker = self
            .depthwise_weights
            .as_slice()
            .expect("depthwise weights must be contiguous");

        let mut output = Array3::<f32>::zeros((batch_size, g.output.1, g.out_channels()));
        depthwise_forward(
            &g,
            src,
            ker,
            None,
            output.as_slice_mut().expect("output is contiguous"),
        );

        Ok(output.into_dyn())
    }

    /// Performs the pointwise (1-tap) convolution stage
    ///
    /// A 1-tap convolution is a per-position cross-channel matrix multiply. This delegates to the
    /// shared [`conv_forward`] engine (im2col + gemm). The pointwise weights `[1, C*dm, filters]`
    /// already match the engine's flat `[k..., Cin, F]` layout. The bias is already its
    /// per-filter `[F]` vector. The depthwise stage emits its channels in
    /// `c * depth_multiplier + m` order, which is exactly the row order the pointwise weight
    /// uses. Nothing repacks the data between the stages
    fn pointwise_convolve(&self, input: &Tensor) -> Tensor {
        conv_forward(
            input,
            self.pointwise_weights
                .as_slice()
                .expect("pointwise weights must be contiguous"),
            self.pointwise_weights.shape(),
            self.use_bias
                .then(|| self.bias.as_slice().expect("bias must be contiguous")),
            &[1],
            &[1],
            ConvPadding::Valid,
        )
        // A 1-tap kernel under Valid padding can never exceed the input (every length >= 1)
        .expect("1-tap pointwise convolution geometry is always valid")
    }

    /// Sets whether the layer adds a bias to the pointwise output (defaults to `true`)
    ///
    /// With `use_bias` set to false, the weight roster contains only the 2 kernels. `param_count` counts
    /// them, and `parameters_mut` yields them. Checkpoints store `<position>.depthwise_kernel`
    /// and `<position>.pointwise_kernel`. A checkpoint with bias cannot load into a layer
    /// without bias. The error names the path.
    ///
    /// # Parameters
    ///
    /// - `use_bias` - `true` to add a bias, `false` to leave it out
    ///
    /// # Returns
    ///
    /// - `Self` - The updated layer
    pub fn with_use_bias(mut self, use_bias: bool) -> Self {
        self.use_bias = use_bias;
        self
    }

    /// Sets the weights and bias for this layer
    ///
    /// # Parameters
    ///
    /// - `depthwise_weights` - 3D array for depthwise filters with shape
    ///   \[kernel_size, channels, depth_multiplier\]
    /// - `pointwise_weights` - 3D array for pointwise filters with shape
    ///   \[1, channels * depth_multiplier, filters\]
    /// - `bias` - 1D bias vector with shape \[filters\], or `None` for a layer built with
    ///   [`with_use_bias(false)`](Self::with_use_bias)
    ///
    /// # Returns
    ///
    /// - `Result<(), Error>` - Success after the weights are set
    ///
    /// # Errors
    ///
    /// - `Error::NeuralNetwork(NnError::WeightShape)` - If any supplied array shape does not
    ///   match the existing layer weights
    /// - `Error::InvalidParameter` - If a bias is given to a layer that holds none, or none
    ///   is given to a layer that holds 1
    pub fn set_weights(
        &mut self,
        depthwise_weights: Array3<f32>,
        pointwise_weights: Array3<f32>,
        bias: impl Into<Option<Array1<f32>>>,
    ) -> Result<(), Error> {
        if self.built.is_none() {
            return Err(Error::not_built("SeparableConv1D"));
        }
        validate_weight_shape(
            "depthwise_kernel",
            self.depthwise_weights.shape(),
            depthwise_weights.shape(),
        )?;
        validate_weight_shape(
            "pointwise_kernel",
            self.pointwise_weights.shape(),
            pointwise_weights.shape(),
        )?;
        let bias = validate_optional_weight("bias", "use_bias", self.use_bias, bias.into())?;
        if let Some(bias) = bias.as_ref() {
            validate_weight_shape("bias", self.bias.shape(), bias.shape())?;
        }
        self.depthwise_weights = depthwise_weights;
        self.pointwise_weights = pointwise_weights;
        if let Some(bias) = bias {
            self.bias = bias;
        }
        Ok(())
    }
}

/// What the forward pass of [`SeparableConv1D`] parks for its backward pass
struct SeparableConv1DCache {
    /// The tensor the forward pass received
    input: Tensor,
    /// The output of the depthwise stage, which the pointwise backward pass reads
    depthwise_output: Tensor,
    /// The tensor that the activation backward pass reads, as [`Activation::saves`] names
    activation: ActivationCache,
}

impl LayerBase for SeparableConv1D {
    fn layer_type(&self) -> &str {
        "SeparableConv1D"
    }

    fn param_count(&self) -> ParamCounts {
        // Read the arrays the layer holds rather than the configuration, so dropping the
        // bias corrects the count with no second formula to keep in step
        let bias = if self.use_bias { self.bias.len() } else { 0 };
        ParamCounts::trainable(self.depthwise_weights.len() + self.pointwise_weights.len() + bias)
    }

    fn parameters_mut(&mut self) -> Vec<ParamRef<'_>> {
        let Self {
            depthwise_weights,
            pointwise_weights,
            bias,
            use_bias,
            ..
        } = self;
        let mut params = vec![
            ParamRef::weight(
                "depthwise_kernel",
                depthwise_weights
                    .as_slice_mut()
                    .expect("depthwise weights must be contiguous"),
            ),
            ParamRef::weight(
                "pointwise_kernel",
                pointwise_weights
                    .as_slice_mut()
                    .expect("pointwise weights must be contiguous"),
            ),
        ];
        if *use_bias {
            params.push(ParamRef::no_decay(
                "bias",
                bias.as_slice_mut().expect("bias must be contiguous"),
            ));
        }
        params
    }

    built_layer_shape_functions!();

    named_weight_layer_functions!(
        trainable "depthwise_kernel" => depthwise_weights,
        trainable "pointwise_kernel" => pointwise_weights,
        trainable "bias" => bias if use_bias,
    );
}

impl UnaryLayer for SeparableConv1D {
    /// Reads the channel count from the input shape, and draws both kernels and the bias
    ///
    /// 1 generator threads the depthwise draw and then the pointwise draw, in that order. A
    /// second generator, or the other order, changes every value of the second draw
    fn build(&mut self, input: &Shape) -> Result<(), Error> {
        let Some(built) = start_build(&self.built, "SeparableConv1D", input)? else {
            return Ok(());
        };
        built.check_rank("SeparableConv1D", 3)?;
        let (batch, tail) = built.split_batch("SeparableConv1D")?;
        // The family validators read a full extent list, and the batch extent is not part of
        // what they check
        let mut dims = vec![batch.unwrap_or(1)];
        dims.extend(tail);
        validate_input_shape_1d(&dims)?;
        // The shape algebra holds every rule the geometry has, so a stack that cannot run is
        // refused here, before the layer draws a single weight
        self.compute_output_shape(&built)?;
        self.channels = dims[2];
        self.built = Some(built);
        self.draw_parameters();
        Ok(())
    }

    fn forward(&self, input: &Tensor, ctx: &mut Ctx) -> Result<Tensor, Error> {
        if !self.is_built() {
            return Err(Error::not_built("SeparableConv1D"));
        }
        self.validate_input(input)?;

        let depthwise_output = self.depthwise_convolve(input)?;
        let output = self.pointwise_convolve(&depthwise_output);

        if !ctx.is_training() {
            return self.activation.forward(&output);
        }
        let (activated, activation) = self.activation.forward_train(output)?;

        // Park only after a successful pass, so a rejected input leaves no partial state. The
        // depthwise output moves into the cache with no copy
        ctx.push_cache(
            "SeparableConv1D",
            SeparableConv1DCache {
                input: input.clone(),
                depthwise_output,
                activation,
            },
        );
        Ok(activated)
    }

    fn backward(&self, grad_output: &Tensor, ctx: &mut Ctx) -> Result<Tensor, Error> {
        let cache: SeparableConv1DCache = ctx.pop_cache("SeparableConv1D")?;

        let grad_upstream = self.activation.backward(&cache.activation, grad_output)?;

        let input = &cache.input;
        let depthwise_output = &cache.depthwise_output;

        let batch_size = input.shape()[0];
        let g = self.depthwise_geometry(input.shape())?;

        // Pointwise (1-tap) backward via the shared engine (im2col + gemm). Its input gradient is
        // the gradient with respect to the depthwise output, with shape [batch, L', C*dm]
        let pw_grads = conv_backward(
            &grad_upstream,
            depthwise_output,
            self.pointwise_weights
                .as_slice()
                .expect("pointwise weights must be contiguous"),
            self.pointwise_weights.shape(),
            &[1],
            &[1],
            ConvPadding::Valid,
        )
        // 1-tap Valid geometry is always valid (see `pointwise_convolve`)
        .expect("1-tap pointwise convolution geometry is always valid");
        ctx.add_grad(
            "pointwise_kernel",
            Array3::from_shape_vec(self.pointwise_weights.raw_dim(), pw_grads.weight_grad)
                .expect("pointwise weight gradient shape matches weights")
                .into_dyn(),
        )?;
        // A layer without bias has no stored bias gradient or bias optimizer state.
        if self.use_bias {
            ctx.add_grad("bias", Array1::from_vec(pw_grads.bias_grad).into_dyn())?;
        }
        let depthwise_grad = pw_grads.input_grad;

        let input_std = input.as_standard_layout();
        let src = input_std
            .as_slice()
            .expect("standard-layout array is contiguous");
        let grad_std = depthwise_grad.as_standard_layout();
        let grad = grad_std
            .as_slice()
            .expect("standard-layout array is contiguous");
        let ker = self
            .depthwise_weights
            .as_slice()
            .expect("depthwise weights must be contiguous");

        // This stage has no bias of its own, so backward drops the bias field the shared driver
        // returns
        let dw_grads = depthwise_backward(&g, src, grad, ker, batch_size);

        ctx.add_grad(
            "depthwise_kernel",
            Array3::from_shape_vec(self.depthwise_weights.raw_dim(), dw_grads.weight)
                .expect("depthwise weight gradient shape matches weights")
                .into_dyn(),
        )?;

        Ok(
            Array3::from_shape_vec((batch_size, g.input.1, g.channels), dw_grads.input)
                .expect("input gradient shape matches input")
                .into_dyn(),
        )
    }

    fn compute_output_shape(&self, input: &Shape) -> Result<Shape, Error> {
        input.check_rank("SeparableConv1D", 3)?;
        let (batch, tail) = input.split_batch("SeparableConv1D")?;
        Ok(Shape::from_batch(
            batch,
            &[self.calculate_output_length(tail[0])?, self.filters],
        ))
    }
}

/// Unit tests for `SeparableConv1D`
#[cfg(test)]
mod tests {
    use super::*;
    use crate::neural_network::layers::activation::linear::Linear;
    use ndarray::ArrayD;

    /// The 2 stages agree on the intermediate channel order
    ///
    /// A 1-tap kernel at a single position reduces the layer to arithmetic a reader can write out
    /// by hand. The depthwise weights give each `(channel, multiplier)` pair a distinct power of
    /// 10. The pointwise weights give each intermediate channel a distinct power of 2. The single
    /// output value only comes out right if the depthwise stage emits its channels in
    /// `c * depth_multiplier + m` order. It also needs the pointwise weight rows indexed in that
    /// same order. Any transposition of either would change the total
    #[test]
    fn separable_1d_stage_channel_order_hand_derived() {
        let mut layer = SeparableConv1D::new(1, 1, 1, 2, Linear::new()).unwrap();
        layer.build(&Shape::known(&[1, 1, 2])).unwrap();
        assert_eq!(layer.depthwise_weights.shape(), &[1, 2, 2]);
        assert_eq!(layer.pointwise_weights.shape(), &[1, 4, 1]);

        // [k, C, dm] as (c, m): c0 -> [1, 10], c1 -> [100, 1000]
        let depthwise = Array3::from_shape_vec((1, 2, 2), vec![1.0, 10.0, 100.0, 1000.0]).unwrap();
        // [1, C*dm, F]: 1 distinct weight per intermediate channel
        let pointwise = Array3::from_shape_vec((1, 4, 1), vec![1.0, 2.0, 4.0, 8.0]).unwrap();
        layer
            .set_weights(depthwise, pointwise, Array1::zeros(1))
            .unwrap();

        // 1 position holding [2, 3]
        let input = ArrayD::from_shape_vec(ndarray::IxDyn(&[1, 1, 2]), vec![2.0, 3.0]).unwrap();
        let out = layer.forward(&input, &mut Ctx::inference()).unwrap();

        // Intermediate = [2*1, 2*10, 3*100, 3*1000] = [2, 20, 300, 3000]
        // Output = 2*1 + 20*2 + 300*4 + 3000*8 = 25242
        assert_eq!(out.shape(), &[1, 1, 1]);
        assert_eq!(out.iter().copied().collect::<Vec<f32>>(), vec![25242.0]);
    }

    /// Along the length axis, the depthwise stage is an ordinary per-channel cross-correlation.
    /// The pointwise stage is a per-position channel mix. A width-2 kernel over a length-4 input
    /// reduces to the 3 window sums, scaled by the pointwise weight
    #[test]
    fn separable_1d_length_pass_hand_derived() {
        let mut layer = SeparableConv1D::new(1, 2, 1, 1, Linear::new()).unwrap();
        layer.build(&Shape::known(&[1, 4, 1])).unwrap();

        // Single channel, all-ones depthwise kernel. Pointwise scales by 3
        let depthwise = Array3::from_shape_vec((2, 1, 1), vec![1.0, 1.0]).unwrap();
        let pointwise = Array3::from_shape_vec((1, 1, 1), vec![3.0]).unwrap();
        layer
            .set_weights(depthwise, pointwise, Array1::zeros(1))
            .unwrap();

        // [1, 4, 1] holding 1..4
        let input =
            ArrayD::from_shape_vec(ndarray::IxDyn(&[1, 4, 1]), vec![1.0, 2.0, 3.0, 4.0]).unwrap();
        let out = layer.forward(&input, &mut Ctx::inference()).unwrap();

        // Window sums 3, 5, 7, each tripled
        assert_eq!(out.shape(), &[1, 3, 1]);
        assert_eq!(
            out.iter().copied().collect::<Vec<f32>>(),
            vec![9.0, 15.0, 21.0]
        );
    }
}