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

Midi Merge / Midi Thru introduce noticebale latency #31

Closed
garubi opened this issue Oct 29, 2018 · 4 comments
Closed

Midi Merge / Midi Thru introduce noticebale latency #31

garubi opened this issue Oct 29, 2018 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@garubi
Copy link
Owner

garubi commented Oct 29, 2018

No description provided.

@garubi garubi added the bug Something isn't working label Oct 29, 2018
@garubi garubi changed the title Midi Merge / Midi Thru itroduce noticeale latency Midi Merge / Midi Thru introduce noticeale latency Oct 29, 2018
@garubi garubi changed the title Midi Merge / Midi Thru introduce noticeale latency Midi Merge / Midi Thru introduce noticebale latency Oct 30, 2018
@garubi
Copy link
Owner Author

garubi commented Oct 30, 2018

Probably we should refactor the code in the loop, but first we could try some things:

  • check if all that SerialPrint have something to do with it
    -- try faster baudrate
    -- remove all the serialprint() (maybe use something like
    `#define DEBUG false

// conditional debugging
#if DEBUG

#define beginDebug() do { Serial.begin (115200); } while (0)
#define Trace(x) Serial.print (x)
#define Trace2(x,y) Serial.print (x,y)
#define Traceln(x) Serial.println (x)
#define Traceln2(x,y) Serial.println (x,y)
#define TraceFunc() do { Serial.print (F("In function: ")); Serial.println (PRETTY_FUNCTION); } while (0)

#else
#define beginDebug() ((void) 0)
#define Trace(x) ((void) 0)
#define Trace2(x,y) ((void) 0)
#define Traceln(x) ((void) 0)
#define Traceln2(x,y) ((void) 0)
#define TraceFunc() ((void) 0)
#endif // DEBUG`

@garubi
Copy link
Owner Author

garubi commented Nov 5, 2018

maybe try to enable running status in the MIDI library

https://github.com/FortySevenEffects/arduino_midi_library/wiki/Using-custom-Settings

@garubi
Copy link
Owner Author

garubi commented Nov 7, 2018

lot of interesting info about the midi ibrary here:
FortySevenEffects/arduino_midi_library#71

@garubi
Copy link
Owner Author

garubi commented Nov 17, 2018

Greatly reduced the latency.
It's well playable with piano and fast-attack sounds too.
We can close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant