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 Mar 1, 2024
1 parent d5a8e3a commit 96f9564
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 96f9564

Please sign in to comment.