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

Fix problems with knobs values restore #771

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

AndyMcFishka
Copy link
Contributor

@AndyMcFishka AndyMcFishka commented Jan 21, 2024

-get rid of duplicated refresh midi controller code
-fix problem with knob values restore at profile switch
-higher priority to alt param value
-nil checks fixed

@AndyMcFishka
Copy link
Contributor Author

related to #756 and #762 (comment)

@AndyMcFishka AndyMcFishka changed the title Fix problems with knobs values restore [WIP] Fix problems with knobs values restore Jan 21, 2024
@AndyMcFishka AndyMcFishka marked this pull request as draft January 21, 2024 02:52
-get rid of duplicated refresh midi controller code
-fix problem with knob values restore at profile switch
-higher priority to alt param value
-nil checks fixed
@AndyMcFishka AndyMcFishka force-pushed the fix-knobs-restore-values branch from ee9456c to 2c79fb9 Compare January 21, 2024 04:21
@AndyMcFishka AndyMcFishka marked this pull request as ready for review January 21, 2024 04:23
@AndyMcFishka AndyMcFishka changed the title [WIP] Fix problems with knobs values restore Fix problems with knobs values restore Jan 21, 2024
@Darek24Szubi
Copy link

Super information!

Please let me know the safest way to make these changes?
Is there any way to swap files simply and it will be ok?

OK - I've swapped files and everything looks promising so far - thanks!

@AndyMcFishka
Copy link
Contributor Author

Super information!

Please let me know the safest way to make these changes? Is there any way to swap files simply and it will be ok?

OK - I've swapped files and everything looks promising so far - thanks!

Glad to hear that it fixed the problem for you. For everybody with the same question - yes, you can replace files and restart plugin, thats all

Copy link
Owner

@rsjaffe rsjaffe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your fixes. Will merge and then refactor as local work not yet pushed to github has refactored FullRefresh to reduce dependencies and get rid of duplicate function.

@rsjaffe rsjaffe merged commit 4605af1 into rsjaffe:develop Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants