M-Labs
/
thermostat
9
0
Fork 5
Code Issues 9 Pull Requests 5 Releases Wiki Activity

Calculate current_abs_max_tec_i from all channels #119

Merged
sb10q merged 1 commits from atse/thermostat:generalise-fan-channel into master 2024-08-07 16:14:29 +08:00
Conversation 0 Commits 1 Files Changed 1 +5 -4
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5637c0df62 - Show all commits

9
src/channels.rs
View File

@ -1,4 +1,4 @@
use core::{cmp::max_by, marker::PhantomData};
use core::marker::PhantomData;
use heapless::{consts::U2, Vec};
use num_traits::Zero;
use serde::{Serialize, Serializer};
@ -561,9 +561,10 @@ impl Channels {
}
pub fn current_abs_max_tec_i(&mut self) -> ElectricCurrent {
max_by(self.get_tec_i(0).abs(),
self.get_tec_i(1).abs(),
|a, b| a.partial_cmp(b).unwrap_or(core::cmp::Ordering::Equal))
(0..CHANNELS)
.map(|channel| self.get_tec_i(channel).abs())
.max_by(|a, b| a.partial_cmp(b).unwrap_or(core::cmp::Ordering::Equal))
esavkin commented 2024-08-07 10:22:55 +08:00
Review
Copy Link

Quite curious though, is it internally same efficiency? I remember in some languages it's not in case of streams

Quite curious though, is it internally same efficiency? I remember in some languages it's not in case of streams
linuswck commented 2024-08-07 11:14:16 +08:00
Review
Copy Link

It doesn't matter here. The bottleneck will be self.get_tec_i(channel) which blocks until the single conversion Adc readings return.

It doesn't matter here. The bottleneck will be `self.get_tec_i(channel)` which blocks until the single conversion Adc readings return.
.unwrap()
}
}