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

Make toggling/+/- work for "MIDI/OSC: Send message" targets #387

Closed
helgoboss opened this issue Jun 17, 2021 · 3 comments
Closed

Make toggling/+/- work for "MIDI/OSC: Send message" targets #387

helgoboss opened this issue Jun 17, 2021 · 3 comments
Labels
easy Would be quite easy to implement enhancement New feature or request high priority realearn Related to ReaLearn

Comments

@helgoboss
Copy link
Owner

Should be easy by letting these targets report an artifical current value - then "Incremental buttons" and "Toggle buttons" modes should just work.

@helgoboss helgoboss added the enhancement New feature or request label Jun 17, 2021
@helgoboss helgoboss added high priority easy Would be quite easy to implement labels Jul 15, 2021
@helgoboss
Copy link
Owner Author

helgoboss commented Jul 17, 2021

This should also fix non-working "Reverse".

Wrong: The non-working "Reverse" is a regression because of 2.10.0-pre3 #184 preparations that affects discrete targets.

@helgoboss
Copy link
Owner Author

Problem with the "artificial current value" approach: It's one value per mapping. That makes it not suitable for "Incremental buttons" scenarios, which needs 2 mappings.

@helgoboss
Copy link
Owner Author

helgoboss commented Jul 17, 2021

For now that's enough. This change adds the following possibilities, which is already a nice improvement:

  • "Toggle buttons" mode works as expected.
  • "Incremental buttons" mode works, but in one direction only (because 2nd mapping has another current value and therefore can't participate).
  • Control with encoders works without having to tick "Make absolute" which should avoid some "Why!!!?" moments. The effect is the same.

I attempted sharing current values within one ReaLearn instance, qualified by target settings. But there are issues:

  1. Keeping the target settings as key in a map requires allocation, which is a no-go at least in the audio thread (important for real-time MIDI conversion).
  2. It's possible that the user wants to share one current value even if the target pattern is slightly different.

The cleanest thing to do would be to introduce a kind of group index within which the current value is shared. But this can still be done in future.

helgoboss added a commit that referenced this issue Jul 17, 2021
Works but has a practical issue: Because it's a current value per mapping,
setting up incremental buttons with multiple mappings doesn't work as expected.
@helgoboss helgoboss added the realearn Related to ReaLearn label Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy Would be quite easy to implement enhancement New feature or request high priority realearn Related to ReaLearn
Projects
None yet
Development

No branches or pull requests

1 participant