teotigraphix / Framework4Bitwig

Bitwig controller API framework extensions.
GNU Lesser General Public License v3.0
58 stars 13 forks source link

CursorClip Update buggy #247

Closed git-moss closed 4 years ago

git-moss commented 4 years ago

There is an update bug with CursorClip.addNoteStepObserver (this::handleStepData). To reproduce use the Push note sequencer: 1) create track 2) create 1bar clip 3) put in notes in note sequencer position 1 and 5 on 1st page 4) witch to second page and put note in position 1 5) position 5 will light up on Push even if there is no note in the corresponding position inside clip. 6) Switch back to page 1, looks good 7) Switch to page 2, also looks good now

Debugging shows that when the note on the 2nd page is created, notes are sent starting from the 1st page (incl. the 2nd). CursorClip Update Bug

git-moss commented 4 years ago

Still in 3.1.3.

abique commented 4 years ago

Hey Moss, I don't have push, but I have the launchpad pro (mk1). I've tried your steps with the Bitwig extension and I could not reproduce it. Could it be a bug in your implementation?

git-moss commented 4 years ago

I fixed it by removing the arrangerClip. Not sure why it did not work when we tested it.