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

Convenience feature for snapshots: Add "Reverse" or "Unmute" checkbox for mute target #630

Closed
helgoboss opened this issue Jul 12, 2022 · 2 comments
Labels
enhancement New feature or request high priority realearn Related to ReaLearn

Comments

@helgoboss
Copy link
Owner

"Unmute" is a quite popular use case because it's used to save CPU power. While one can use "Reverse" in the Glue section, it's not usable for snapshot loading with default values, which changes the target directly.

@helgoboss helgoboss added enhancement New feature or request low priority labels Jul 12, 2022
@helgoboss
Copy link
Owner Author

helgoboss commented Jul 14, 2022

Potentially useful for other on/off targets as well - whenever the "Glue" section is skipped:

  1. Target "ReaLearn: Load mapping snapshot": Relevant for the default value
  2. Target "ReaLearn: Navigate within group"
  • Mmh, but the "Glue" section is not completely skipped, target min/max is taken into account!
  1. Group interaction with "Same target value" or "Inverse target value"
  • Okay, but there's the alternative "Same control" or "Inverse control" already ... so there's no real benefit there.

So ... one way would be to change behavior a bit and respect the "Reverse" setting for 1 and 2. I prefer that. Existing users are unlikely to be affected because:

  • "ReaLearn: Load mapping snapshot" with real memorized snapshots is brand-new and rarely used. It's also only relevant for snapshot targets with default value that don't cover all matching mappings.
  • "ReaLearn: Navigate within group" exists longer, but it's unlikely that users assign the navigated mappings also to sources AND have the reverse checkbox checked.

@helgoboss
Copy link
Owner Author

1 (load mapping snapshot respecting reverse) is now implemented. It's applied to the default value only. Existing snapshots of the mapping are unaffected (which is important for backwards compatibility).

2 (group interaction respecting reverse) has been implemented in commit b10cb8a already.

@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
enhancement New feature or request high priority realearn Related to ReaLearn
Projects
None yet
Development

No branches or pull requests

1 participant