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

Good job #6

Open
Aepelzen opened this issue Jul 7, 2019 · 3 comments
Open

Good job #6

Aepelzen opened this issue Jul 7, 2019 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@Aepelzen
Copy link

Aepelzen commented Jul 7, 2019

Hi,

good job, these look really nice. The only minor thing I spotted on a quick first inspection is the big knob on the retrig module. It's label says pulsewidth but it is really more like the samplerate. It sets the time-interval at which the input signal is sampled and compared to it's previous value.

I also have some tips and things I discovered while doing my own port in case you are interested:

  • the samplerate-converter apparently does not like non-integers ratios. You can improve the refold performance quite drastically by restricting the upsample-ratio to integer values
  • The mixer performance can also easily be improved. You can skip channels that are not connected, so you don't have to process the equalizers unnecessarily. Replacing the vu-meter with the new version also helped a lot if I remember correctly. Those simple changes reduced the idle cpu-load (when nothing is connected) from 5% to 1% on my machine
  • if you plan to include the sampler in later stages, I recommend putting the sample-display into a buffered widget. That improves performance and also fixes the crashing problems

That's it for now. If you have any questions or need help figuring out how some things work, let me know.

@wiqid
Copy link
Owner

wiqid commented Jul 7, 2019

thanks so much! i will certainly look into your tips.

we do plan on including the sequencers, but not the sampler.

@wiqid wiqid self-assigned this Jul 7, 2019
@wiqid wiqid added the enhancement New feature or request label Jul 7, 2019
@ShofB
Copy link

ShofB commented Jul 8, 2019

I wish you would reconsider on the sampler. I have found it very useful.

@wiqid
Copy link
Owner

wiqid commented Jul 8, 2019

the sampler was an unfinished module with known issues. i do not have a particular interest in it. between the nysthi samplers and m80 radio music i think our sample needs are covered.

anyone who wants to pick it up and develop it further can of course do so. the joys of open source!

alternatively, you could ask nysthi's dev to implement features you are missing. he is quite open to requests and quick to respond.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants