Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • P purr-data
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 363
    • Issues 363
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 54
    • Merge requests 54
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Jonathan Wilkes
  • purr-data
  • Merge requests
  • !77

Merged
Created Feb 13, 2017 by Albert Gräf@aggraefDeveloper

Make sure that inmidi_clk() is called in the right place in the MIDI back-end. Fixes #255.

  • Overview 3
  • Commits 2
  • Changes 1

Remarks:

  • Note that I didn't fiddle with the implementation of the midiclkin object itself, which seems to be all right. I just made sure that it gets triggered in the appropriate place.

  • There isn't any useful documentation whatsoever for this object, so it's up to anyone's guess what the timebase for the time deltas in the right outlet should be. I've chosen msecs for now, which is in line with the default unit used by metro, delay, etc.

  • There is noticeable jitter in the time diff values, but that's to be expected with the simplistic implementation of midiclkin which just reports the deltas as they are. So in order to actually do some professional-grade tempo tracking or syncing with this thing, the user still has to think about some appropriate averaging process -- I'm not an expert on this, but I've been told that one can use PLLs = phase locked loops for that kind of thing.

Ok, there you have it. At least the object now does something sensible. :)

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: midiclkin-fix