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

Fuse Rhythm to Melody #7618

Closed
wants to merge 6 commits into from
Closed

Fuse Rhythm to Melody #7618

wants to merge 6 commits into from

Conversation

regulus79
Copy link
Contributor

@regulus79 regulus79 commented Dec 9, 2024

Decription

This feature allows the user to select two sets of clips on different tracks; one with the rhythm, and one with the melody. Then, by rightclicking on one of the melody clips, they can select "Fuse Rhythm to Melody," which will replace the old melody clips with a new clip which contains all the same notes, but while the keys of the notes are taken from the melody clips, the positions, velocity, and panning are taken from the rhythm clips.

Based on #7126

2024-12-09.14-55-47.mp4

Screenshot From 2024-12-09 14-58-07

Copy link

@xor32dotcom xor32dotcom left a comment

Choose a reason for hiding this comment

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

i second this commit as i came up with the idea behind the feature as mentioned on the merge description.

Apparently it has a bug or two but this is 100% in at least a partial usable state for a nightly release

@regulus79
Copy link
Contributor Author

What bugs have you been encountering?

@xor32dotcom
Copy link

came up with the best instrument to reuse this code if needed
"transcriptor"

@tresf
Copy link
Member

tresf commented Feb 19, 2025

This is a cool feature. Embarrassingly I'm not exactly sure how to use it, but I'll offer some early feedback.

  • I think any feature that automagically uses a source and target should have some hand-holding. For example, "Right click on the melody" is a bit ambiguous. If the context menu could help explain what the action is going to do (e.g. "Fuse Noisy Guitar (Rhythm) to Horn (Melody)") this might be a bit easier to understand the flow of.
  • The fact that this context menu item is missing unless two separate track patterns are selected makes it a bit hard to stumble upon this feature. It may make sense to offer it grayed-out with some help text for the user to discover on their own.
  • The length acts weird sometimes. I'm not sure why.

Here's some very basic testing:
https://youtu.be/MZn1b-_eSbc

@regulus79
Copy link
Contributor Author

Thank you for testing this!

However, michealgregorius gave some feedback in #7126 regarding this feature which makes me want to potentially hold off on this PR. The way this feature works with selections and rightclicking is not ideal. This sort of feature is only really useful for a few people in very rare cases, and having it in the rightclick menu does not seem appropriate. I think it may be better to wait until midi routing and midi effect support is implemented to add this feature. But I would love to hear everyone else's thoughts too.

@tresf
Copy link
Member

tresf commented Feb 20, 2025

@regulus79 what would you like to do? Would you like to close this PR now and reference it at a later time?

@regulus79
Copy link
Contributor Author

If we are trying to archive stale/dormant PRs, I think that may be the right choice.

@regulus79 regulus79 closed this Feb 20, 2025
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