adafruit-mirror/Adafruit_Blinka_Raspberry_Pi5_Piomatter
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tweak comment & no-temporal check

Browse source
This commit is contained in:
Jeff Epler 2025-03-10 08:24:50 -05:00
parent 4f548a318c
commit e2c5bc3467
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/include/piomatter/matrixmap.h
View file

@ -104,7 +104,7 @@ schedule_sequence make_simple_schedule(int n_planes, size_t pixels_across) {
} }
// Make a temporal dither schedule. All the top `n_planes` are shown everytime, // Make a temporal dither schedule. All the top `n_planes` are shown everytime,
// but the lowest frames are done in a cycle of `n_temporal_frames`: // but the lowest planes are done in a cycle of `n_temporal_planes`:
// 2: {0, 1}; 4: {0, 1, 2, 3} // 2: {0, 1}; 4: {0, 1, 2, 3}
schedule_sequence make_temporal_dither_schedule(int n_planes, schedule_sequence make_temporal_dither_schedule(int n_planes,
size_t pixels_across, size_t pixels_across,
@ -112,7 +112,8 @@ schedule_sequence make_temporal_dither_schedule(int n_planes,
if (n_planes < 1 || n_planes > 10) { if (n_planes < 1 || n_planes > 10) {
throw std::range_error("n_planes out of range"); throw std::range_error("n_planes out of range");
} }
if (n_temporal_planes == 0) { if (n_temporal_planes < 2) {
// either 0 or 1 temporal planes are not really temporal at all
return make_simple_schedule(n_planes, pixels_across); return make_simple_schedule(n_planes, pixels_across);
} }
if (n_temporal_planes >= n_planes) { if (n_temporal_planes >= n_planes) {