Skip to content

Commit 59d9e17

Browse files
committed
refactor: improve code quality and remove unused imports
- Use if-let pattern for cleaner option handling in direct_solver - Remove unused imports from window wgpu tests
1 parent a59ce82 commit 59d9e17

2 files changed

Lines changed: 1 addition & 6 deletions

File tree

src/integrate/impl_generic/ode/direct_solver.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -193,10 +193,8 @@ impl<R: Runtime> DirectSparseSolver<R> {
193193
// Apply equilibration to cached CSC if enabled
194194
let factored_csc = if self.equilibrate {
195195
let csc_ref = self.cached_permuted_csc.as_ref().unwrap();
196-
if self.row_scales.is_some() {
196+
if let (Some(row_scales), Some(col_scales)) = (&self.row_scales, &self.col_scales) {
197197
// Reuse cached scales
198-
let row_scales = self.row_scales.as_ref().unwrap();
199-
let col_scales = self.col_scales.as_ref().unwrap();
200198
let scaled = csc_ref.scale_rows(row_scales)?;
201199
scaled.scale_cols(col_scales)?
202200
} else {

src/window/wgpu/window.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,9 +104,6 @@ impl WindowFunctions<WgpuRuntime> for WgpuClient {
104104

105105
#[cfg(test)]
106106
mod tests {
107-
use super::*;
108-
use numr::dtype::DType;
109-
110107
#[test]
111108
#[ignore] // WebGPU tests require GPU
112109
fn test_hann_window_wgpu() {

0 commit comments

Comments
 (0)