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

Remove the saving midi connection option completely. #7442

Closed
wants to merge 3 commits into from

Conversation

Rossmaxx
Copy link
Contributor

Fixes #3922 I believe.

If not, please guide me to a fix.

@Rossmaxx
Copy link
Contributor Author

@PhysSong @zonkmachine need some help here

@michaelgregorius
Copy link
Contributor

michaelgregorius commented Aug 11, 2024

Now that I have checked #3922 again I am not really sure if this is really the right way to fix the problem that's reported in that issue.

My understanding is that this features enables users to save their MIDI connections in project files so that they can immediately use their MIDI hardware when loading such a file later. If this pull request goes through they'd have to reconnect their devices over and over after each load which would be quite tedious.

The solution to #3922 would be to prevent the storage of MIDI connections when a preset is saved but to keep saving them when a project is being saved.

Edit: I assume that "Don't save MIDI settings." was intended to mean "Don't save MIDI settings when saving presets." in #3922.

@Rossmaxx
Copy link
Contributor Author

Rossmaxx commented Aug 11, 2024

@michaelgregorius your explanation makes sense too, can you tell me which lines to look at if i am to disable the saving for presets instead?

Can someone else clarify michael's assumption?

@michaelgregorius
Copy link
Contributor

@michaelgregorius your explanation makes sense too, can you tell me which lines to look at if i am to disable the saving for presets instead?

Can someone else clarify michael's assumption?

I had started to gather the relevant information for you but then I realized it might be quicker if I simply open a pull request with the things I would have proposed to do. It's pull request #7445.

@Rossmaxx
Copy link
Contributor Author

Okk. I'll close it in favour of #7445

@Rossmaxx Rossmaxx closed this Aug 11, 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.

Instrument presets saves MIDI settings
3 participants