Skip to content

Commit

Permalink
miri pass nn::layers::add_into::tests::longer_network
Browse files Browse the repository at this point in the history
Thread main finished before other threads were still active. The only
thing related to threads were the gemm usage of rayon, and disabling
rayon usage for matmul seems to be sufficient to fix this.
  • Loading branch information
swfsql committed Dec 5, 2023
1 parent 0052366 commit c6e0642
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions dfdx-core/src/tensor_ops/matmul/cpu_kernel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ impl MatMulImpl<crate::dtypes::AMP<half::f16>> for Cpu {
false,
false,
false,
gemm::Parallelism::Rayon(rayon::current_num_threads()),
gemm::Parallelism::None,
)
}
}
Expand Down Expand Up @@ -138,7 +138,7 @@ impl MatMulImpl<half::f16> for Cpu {
false,
false,
false,
gemm::Parallelism::Rayon(rayon::current_num_threads()),
gemm::Parallelism::None,
)
}
}
Expand Down Expand Up @@ -180,7 +180,7 @@ impl MatMulImpl<f32> for Cpu {
false,
false,
false,
gemm::Parallelism::Rayon(rayon::current_num_threads()),
gemm::Parallelism::None,
)
}
}
Expand Down Expand Up @@ -222,7 +222,7 @@ impl MatMulImpl<f64> for Cpu {
false,
false,
false,
gemm::Parallelism::Rayon(rayon::current_num_threads()),
gemm::Parallelism::None,
)
}
}
Expand Down

0 comments on commit c6e0642

Please sign in to comment.