Skip to content

Commit 39a312c

Browse files
committed
apply review feedback
1 parent ee0ddcf commit 39a312c

File tree

2 files changed

+5
-12
lines changed

2 files changed

+5
-12
lines changed

src/distributions/gamma.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -304,9 +304,6 @@ mod test {
304304
use distributions::Distribution;
305305
use super::{Beta, ChiSquared, StudentT, FisherF};
306306

307-
#[cfg(not(miri))] // Miri is too slow
308-
const N: u32 = 1000;
309-
#[cfg(miri)]
310307
const N: u32 = 100;
311308

312309
#[test]

src/distributions/poisson.rs

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -108,20 +108,16 @@ mod test {
108108
use distributions::Distribution;
109109
use super::Poisson;
110110

111-
#[cfg(not(miri))] // Miri is too slow
112-
const N: u32 = 1000;
113-
#[cfg(miri)]
114-
const N: u32 = 100;
115-
116111
#[test]
112+
#[cfg(not(miri))] // Miri is too slow
117113
fn test_poisson_10() {
118114
let poisson = Poisson::new(10.0);
119115
let mut rng = ::test::rng(123);
120116
let mut sum = 0;
121-
for _ in 0..N {
117+
for _ in 0..1000 {
122118
sum += poisson.sample(&mut rng);
123119
}
124-
let avg = (sum as f64) / (N as f64);
120+
let avg = (sum as f64) / 1000.0;
125121
println!("Poisson average: {}", avg);
126122
assert!((avg - 10.0).abs() < 0.5); // not 100% certain, but probable enough
127123
}
@@ -133,10 +129,10 @@ mod test {
133129
let poisson = Poisson::new(15.0);
134130
let mut rng = ::test::rng(123);
135131
let mut sum = 0;
136-
for _ in 0..N {
132+
for _ in 0..1000 {
137133
sum += poisson.sample(&mut rng);
138134
}
139-
let avg = (sum as f64) / (N as f64);
135+
let avg = (sum as f64) / 1000.0;
140136
println!("Poisson average: {}", avg);
141137
assert!((avg - 15.0).abs() < 0.5); // not 100% certain, but probable enough
142138
}

0 commit comments

Comments
 (0)