colin/fdtd-coremem
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Driver: update TODOs

Browse Source
This commit is contained in:
colin
2022-08-23 01:27:03 -07:00
parent 3c30ac33aa
commit 3326acd125
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/coremem/src/driver.rs
View File

@@ -32,7 +32,6 @@ use std::time::Instant;
pub struct Driver<R, S, Stim=DriverStimulusDynVec<R>> { pub struct Driver<R, S, Stim=DriverStimulusDynVec<R>> {
state: S, state: S,
renderer: Arc<MultiRenderer<S>>, renderer: Arc<MultiRenderer<S>>,
// TODO: use Rayon's thread pool?
render_pool: JobPool<S, ()>, render_pool: JobPool<S, ()>,
measurements: Vec<Arc<dyn AbstractMeasurement<S>>>, measurements: Vec<Arc<dyn AbstractMeasurement<S>>>,
stimuli: StimAccess<R, Stim>, stimuli: StimAccess<R, Stim>,
@@ -530,6 +529,7 @@ impl<R: Real, T: DriverStimulus<R> + Send + 'static> StimAccess<R, T> {
let stim = diag.instrument_stimuli(|| { let stim = diag.instrument_stimuli(|| {
let stim = stim.lock().unwrap(); let stim = stim.lock().unwrap();
let opt = stim.optimized_for(meta, step); let opt = stim.optimized_for(meta, step);
// TODO: remove this box. RenderedStimulus is already heap allocated, so no perf diff?
Box::new(opt.as_ref().rendered( Box::new(opt.as_ref().rendered(
meta.time_step().cast(), meta.time_step().cast(),
// TODO: convert this to an integer // TODO: convert this to an integer