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

Added two new devices. Ambisonic Sample Scrubber and E4L Midi Controlled B Format Sampler #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bothelbows
Copy link

E4L Midi Controlled B Format Sampler is Mark Slee's sampler with simple midi functionality added, as well as speed, pitch (with timestretch), and sample start controls.

Ambisonic Sample Scrubber is another B-format/Ambix sample player built with ~groove. This one has speed, pitch, and sample start/length controls, a filter section with dedicated envelope, an amp envelope, and switchable midi control over the different parameters. It's also set up with viewable controls for the Push 2.

Joe Stratton @bothelbows

E4L Midi Controlled B Format Sampler is Mark Slee's sampler with simple midi functionality added, as well as speed, pitch (with timestretch), and sample start controls.

Ambisonic Sample Scrubber is another B-format/Ambix sampler built with ~groove. This one has speed, pitch, and sample start/length controls, a filter section with dedicated envelope, an amp envelope, and switchable midi control over the different parameters.

Joe Stratton @bothelbows
@roddylindsay
Copy link
Member

Awesome work! I think there's some issue with the Ambisonic Sample Scrubber. The CPU is low until I click Midi Note Control to "pitch" and then everything goes haywire, and clicking "off" doesn't change anything. So I would look into what's happening in that subroutine that might trigger a feedback loop or something that would spike CPU...

@bothelbows
Copy link
Author

bothelbows commented May 16, 2021 via email

@mcslee
Copy link
Contributor

mcslee commented May 16, 2021

Big thanks for these Joe - I will try to do a proper full review of these sometime these week. I'd also like to consider making your Midi Controlled sampler the "standard" one and deprecating the existing B-Format Sampler. My long-term hope was always to make that sampler much more MIDI-oriented and similar to Ableton's native Simpler/Sampler. So after I've haven a look I also may suggest renaming this one to "E4L Simpler" or "E4L Sampler" to imply that this is the "default" sampler people should be using, a la Ableton.

Will keep you posted after I've had a chance to dive into the code.

@bothelbows
Copy link
Author

bothelbows commented May 17, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants