Closed tomcombriat closed 1 week ago
Memory usage change @ c8bfcabedf4635d10bcf6622001a6894cf58cfde
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Untested but I think this should be it. I am leaving some comments on things I am not sure about. Will do a more consistent message once testing is done.
Cheers,
Memory usage change @ 91239735b3fd286673d6747124513d2272e7cb03
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Ok, the static
does not seem to make much difference in the example, but I did not try a mock up example yet to see how the Arduino compiler does understand this case.
I corrected a gross mistake in my use of constrain
and starts to sound good. I am still investigating an overflow in the example AudioDelayFeedbackX2
but which also seems to appear in the master branch (?). I also does not sound as good as the example sound which is online, even in master… On-going.
Memory usage change @ ae923f8fc90367c8ac9bc50491ea76dc06c609b6
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
(Small update on this saga, actually, the example itself is having an overflow. This line
del_samps1 = 64+kDelSamps1.next();
have the right term going to the negative. Somehow, I guess this would be reinterpreted right on 8bitters (haven't tested) but this does not sound like the example on 32-bitters. Next time I put myself to it (a bit late now), will try to change that to del_samps1 = 128+kDelSamps1.next();
to see if that is any more closer to the initial closing.
Sorry for the spam, that's also for me remembering…
Memory usage change @ dfc9c9783e597831305c4e0fd19a4041bd9874ab
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Memory usage change @ 4a85239eb0e4915a7102fd835cac946bd32556ea
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Memory usage change @ a311fa3308617e9e55aca30609caa1581d6b58c6
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Memory usage change @ 043d54586fa1911565fa3fbfd6918f62fd55d213
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Ready for me!
I corrected a few delay examples using char
as per #287 , and one example which had an overflow. Docs will need to be regenerated before the next release.
I will merge this, from testing it seems that the default version is identical to the previous one (except for the two static
variables) and the version with int16_t
also works well (it eats a lot of RAM though).
I am using this class quite a bit these days so if anything wrong went through that will be spotted quite fast. Planning to release in a couple of weeks!
Work in progress!
The idea would be to be able to have types accepted by
AudioDelayFeedback
, exactly the way it is done forAudioDelay
.