Compilation fails on Teensy 3.6 #11
-
|
I'm using a Teensy 3.6 with the following platformio configuration: [env:teensy36]
platform = teensy
board = teensy36
framework = arduino
# get edrumulus src, e.g:
# $ cd ~/projects
# $ mkdir edrumulus
# $ cd edrumulus
# $ git clone https://github.com/corrados/edrumulus.git
lib_extra_dirs =
~/projects/edrumulus
build_flags =
-D USB_MIDI_SERIALI think I worked out everything except for the following compile errors with the same code used in Any idea about these Maybe some more USB macros for Teensy need to be enabled for platformio? See USB Features I will open a pull request that builds |
Beta Was this translation helpful? Give feedback.
Replies: 8 comments 57 replies
-
|
I opened pull request #12 that compiles |
Beta Was this translation helpful? Give feedback.
-
Looks like these constants are defined for teensy 4 in https://github.com/PaulStoffregen/cores/blob/master/teensy4/core_pins.h and missing for the teensy 3.x in https://github.com/PaulStoffregen/cores/blob/master/teensy3/core_pins.h |
Beta Was this translation helpful? Give feedback.
-
|
Also tried teensy 4.0 board which builds fine. So looks like teensy 3.6 isn't supported yet. |
Beta Was this translation helpful? Give feedback.
-
|
The Teensy 3.6 has a complete different CPU on the board. I actually did not expect it to compile/run out of the box. The constants which are missing are for the "keeper" functionality which, I guess, is specific to the CPU on the Teensy 4.0. Do you know if there is a specific define for the Teensy 4.0 and 4.1? I only know of this one: TEENSYDUINO. At least we have to put the "keeper disabling code" in a #ifdef for the Teensy 4.0/4.1. How about the other hardware properties: Is the board LED pin the same pin 13 as on the Teensy 4.0? What is the resolution of the ADC? Is it 12 bits? If not, then ADC_MAX_RANGE has to be adjusted. Also the settings like "setResolution", "setAveraging", "setConversionSpeed" and "setSamplingSpeed". Is it the same? Do we have two ADCs on the Teensy 3.6? A lot of open questions :-). Since I do not have the hardware, I am of little help, unfortunately. |
Beta Was this translation helpful? Give feedback.
-
I found this: #ifndef TEENSY_BOARD_DETECTION_H_INCLUDED
#define TEENSY_BOARD_DETECTION_H_INCLUDED
// Teensy-4.x
#if defined(__IMXRT1062__) || defined (ARDUINO_TEENSY40) || defined (ARDUINO_TEENSY41)
#define TEENSY4
#if defined (ARDUINO_TEENSY40)
#define TEENSY4_0
#elif defined (ARDUINO_TEENSY41)
#define TEENSY4_1
#endif
#endif
// Teensy-3.6
#if defined(__MK66FX1M0__)
# define TEENSY3_6
#endif
// Teensy-3.5
#if defined (__MK64FX512__)
#define TEENSY3_5
#endif
#endifAlso see https://github.com/PaulStoffregen/teensy_size/blob/main/minimal_elf.c
Yes, see https://www.pjrc.com/store/teensy36.html
From https://www.pjrc.com/store/teensy36.html:
Not sure, hopefully it's listed on https://www.pjrc.com/store/teensy36.html |
Beta Was this translation helpful? Give feedback.
-
Thanks, I have applied it to the main branch.
Ok, then no changes required.
So, I would recommend to use the same code as Teensy 4.0/4.1 for now and only use 12 bits of the ADC for initial testing. The first step would be to check if the CPU is fast enough anyway...
If found this forum thread: The Teensy 3.6 has 2 ADC's |
Beta Was this translation helpful? Give feedback.
-
Looks like it's a little bit faster than esp32: https://user-images.githubusercontent.com/305679/141693162-88a18d16-0451-4024-89aa-6b0bf82edaca.png If I can test something, performance wise, please let me know. |
Beta Was this translation helpful? Give feedback.
-
I got 6 of those fancy female stereo jack connectors.
What pad should I start with?
|
Beta Was this translation helpful? Give feedback.
-
|
6 inputs should be no problem. I would start with the PD-120 pad since this is already supported in Edrumulus. |
Beta Was this translation helpful? Give feedback.
-
|
As you have written, you want to use Edrumulus together with Ableton. I assume you are using Windows? Do you know if the Ableton drum synthesizer supports positional sensing? |
Beta Was this translation helpful? Give feedback.
-
The snare actually belongs to a friend and turns out it's not a Roland but a Drum-tec Diablo: https://www.drum-tec.com/drum-tec-diabolo-12-x-5-mesh-head-pad-black It's dual triggering and according to him it supports positional sensing but I don't see it explicitly being mentioned on that page. Or is dual triggering the same?
I'll be using Windows 10 and Ableton 11. Ableton drumracks do not have any special positional sensing support that I know of. I will use Superior Drummer 3 and Addictive Drums 2 VSTs for this. I'm personally happy if everything triggers normally, the positional sensing is a bonus. |
Beta Was this translation helpful? Give feedback.
-
|
The Drum-Tec pad should behave similar to a Roland PD-120. So, I hope you can start with that pad setting in Edrumulus. You could also try out the PD-5 and use the Edrumulus PD-8 preset. But be aware that positional sensing is only supported for mesh pads, not for rubber pads. |
Beta Was this translation helpful? Give feedback.
-
Results for avg 1 on Teensy 3.6 @corrados:
LED dims after 2 sec.
LED dims after 2 sec.
LED dims after 2 sec.
LED dims after 2 sec.
LED dims after 2 sec! |
Beta Was this translation helpful? Give feedback.
-
I mean the ADC averaging setting which you already changed for your tests. |
Beta Was this translation helpful? Give feedback.
-
|
@corrados before i start doing any benchmarks, can you also inc a timestamp in these tests? (although it inc a perf hit) |
Beta Was this translation helpful? Give feedback.
-
|
I haven't actually fully understand your request. In the new plotting mode, the ADC input samples are streamed to the PC. Of course, the serial communication is not fast enough and therefore the stream is interrupted. But this is no issue since the noise samples are uncorrelated anyway. In that stream you cannot insert time stamps. Otherwise the plot would not work anymore. |
Beta Was this translation helpful? Give feedback.
-
I made the circuit for a single female jack connector, connected to GPIO38. So I changed line 65 to include GPIO38: // analog pins setup: snare | kick | hi-hat | hi-hat-ctrl | crash | tom1 | ride | tom2
const int analog_pins[] = { 10, 11, 12, 13, 1, 6, 4, 38 };A Roland PD-5 is connected to GPIO38. When I start the monitor it logs lots of numbers, e.g: So I guess that plotter is important here. I'll see if I can get it running in platformio. |
Beta Was this translation helpful? Give feedback.
-
|
I've installed arduplot and it produces a graph with these commands: export PLATFORMIO_MONITOR_DIR=${HOME}/.platformio/penv/lib/python3.7/site-packages/arduplot/
${HOME}/.platformio/penv/bin/pio device monitor -f plotterbut I don't understand the output (and it's very slow):
I also tried the Arduino 1.8.18 Serial Plotter but the values are also jumping all over the place:
|
Beta Was this translation helpful? Give feedback.
-
On that speed testing branch, I only use the very first input. So, you must set the snare input to 38 to pick up the correct input.
That does not look good. The noise is much too high. I guess it is caused by the fact that you are recording from the wrong GPIO input. |
Beta Was this translation helpful? Give feedback.
-
|
Obviously, some ADC configuration of the Teensy 3.6 is not compatible to the Teensy 4.0 which I use... |
Beta Was this translation helpful? Give feedback.
-
|
@thijstriemstra I don't have any electronic drum pads so I can't tell how well it works, but you can try my teensy 3.6 branch. It's not a real fix (I had to right shift 16 bit down to 12 bit), but it's enough to see it trigger. I was able to do some testing with my KT10 and Hydrogen and it worked well enough for testing. |
Beta Was this translation helpful? Give feedback.
-
|
Actually, only right shifting by 4 had this thing triggering due to, I guess, noise. It didn't seem to be a problem when my foot was in contact with the pedal, but just sitting there doing nothing was triggering. I changed it to right shift by 5 and it's better now. So if you find it triggering like crazy, change the 4 in this line to 5. |
Beta Was this translation helpful? Give feedback.
-
|
nice! @jstma could you make the change against the https://github.com/corrados/edrumulus/tree/teensy_adc_noise_testing branch? |
Beta Was this translation helpful? Give feedback.
-
|
@thijstriemstra I can't easily and they don't look like they're needed anyway. I'd say give the branch a try first and see if it works at all for you. |
Beta Was this translation helpful? Give feedback.
I found this:
Also see https://github.com/PaulStoffregen/teensy_size/blob/main/minimal_elf.c
Yes, see h…