Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retrigger works only partially #14

Open
klangmodulator opened this issue Jan 24, 2020 · 3 comments
Open

Retrigger works only partially #14

klangmodulator opened this issue Jan 24, 2020 · 3 comments

Comments

@klangmodulator
Copy link

Tried to use the retrigger feature of the midithing v2.1. Unfortunately the retrigger ON doesn't work in MONO mode, in DUAL mode only on the first voice and in QUAD mode only on first 3 voices.

I think I spotted the reason: in the function MIDICV::playNote() the gate pin is used as an index for the Retrig array, ranging between 2 to 5. In the ProcessTrigger() function in firmware.ino this array is parsed from 1 to GS.NumVoices.

@xustafu
Copy link
Contributor

xustafu commented Jan 24, 2020

Seems like a bug indeed. Checking it, will come back to you the soonest!

@xustafu
Copy link
Contributor

xustafu commented Nov 9, 2021

Completely forgot to adress this issue!!! There was a firmware change but not uploaded, sorry!
Can you please check if Test branch solves your issue?
https://github.com/Befaco/midithing/tree/Test

@klangmodulator
Copy link
Author

Thanks for coming back on it. I tested it in MONO/DUAL/QUAD mode and yes, the retrigger works now as it should! cool

The only thing I had to change was to rename the include file #include <mcp4728.h> to #include <MCP4728.h> so that the sketch compiled on my machine (built with Arduino 1.8.16 on Win10 and I installed the MCP4728 arduino library version 0.1.0 by "hideakitai")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants