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

Release updated modules: Percall, Chopping Kinky, Hexmix VCA, Kickall (v1.1.0 release) #20

Merged
merged 47 commits into from
Jul 9, 2021

Conversation

hemmer
Copy link
Collaborator

@hemmer hemmer commented Jul 6, 2021

Pull request for #19 Release updated modules: Percall, Chopping Kinky, Hexmix VCA, Kickall (v1.1.0 release)

Leaving as a pull request first to allow @AndrewBelt time to review / add comments. I want to write as close to idiomatic Rack code as possible, especially as I am planning to do more modules in future, so any feedback is very welcome.

Includes @martin-lueders changes for polyphony, plus a little tidying up (e.g. using getPolyVoltageSimd to reduce boilerplate).

hemmer and others added 30 commits March 20, 2021 10:30
* use simd for polyphonic logic
* only process CV / lights every N samples
* only up/downsample if needed
* some code cleanup
* Percall strength should be calculated every sample
Made final row behaviour configurable in the menu (as is the case in the hardware).
- only downsample a source wave generated at higher sample rates
- switch to Pade approximation of sin
- new panel SVG from Befaco
- fix LED logic

Moved all chowdsp library code into specific header.
Updated .json
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
martin-lueders and others added 10 commits June 30, 2021 07:58
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
…nstead of 1.0.1.

Signed-off-by: hemmer <915048+hemmer@users.noreply.github.com>
# Conflicts:
#	plugin.json
#	src/ABC.cpp
#	src/DualAtenuverter.cpp
#	src/EvenVCO.cpp
#	src/Mixer.cpp
#	src/Rampage.cpp
#	src/SlewLimiter.cpp
#	src/SpringReverb.cpp
#	src/plugin.hpp
* try older build machine for linux
* try to convert to idomatic Rack code
* tightened up Percall/Hexmix polyphony logic
plugin.json Outdated Show resolved Hide resolved
src/ABC.cpp Show resolved Hide resolved
@hemmer
Copy link
Collaborator Author

hemmer commented Jul 6, 2021

I also want to add to readme, will it be possible to add screenshots to https://library.vcvrack.com/screenshots/Befaco/ ?

### Dual Atenuverter

Based on [Dual Atenuverter](http://www.befaco.org/dual-atenuverter/)

![Dual Atenuverter](https://library.vcvrack.com/screenshots/Befaco/DualAtenuverter.m.png)

@hemmer hemmer requested a review from AndrewBelt July 6, 2021 12:21
Copy link
Member

@AndrewBelt AndrewBelt left a comment

Choose a reason for hiding this comment

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

I've committed a couple fixes to the VCVRack/Befaco hemmer-v1 branch. Pull and push to that branch for future commits.

The reviewed lines shouldn't take too long to fix. The important points I'd like:

  • Initialize all variables (local and class fields), especially float_4
  • Remove explicit float_4(...) casts, just rely on implicit casts. Well-written SIMD code should look almost exactly like float code, so no need to pollute with unnecessary casts/lines.

src/EvenVCO.cpp Outdated Show resolved Hide resolved
src/EvenVCO.cpp Outdated Show resolved Hide resolved
src/HexmixVCA.cpp Outdated Show resolved Hide resolved
src/Percall.cpp Outdated Show resolved Hide resolved
src/Percall.cpp Outdated Show resolved Hide resolved
src/SlewLimiter.cpp Outdated Show resolved Hide resolved
src/SpringReverb.cpp Show resolved Hide resolved
src/SpringReverb.cpp Outdated Show resolved Hide resolved
.github/workflows/build-plugin.yml Outdated Show resolved Hide resolved
src/EvenVCO.cpp Outdated Show resolved Hide resolved
* use of string::f
* removed unnecessary casts to float_4
* corrected/updated brace initialisation
* removed github actions
* fixing behavior of red light in reverb VU meter
* removed more unnecessary casts to float_4
@hemmer hemmer mentioned this pull request Jul 9, 2021
@AndrewBelt AndrewBelt merged commit 3b80c5c into VCVRack:v1 Jul 9, 2021
@AndrewBelt
Copy link
Member

Looks good. Played with this a bit, felt like I was using my own Eurorack!

@hemmer hemmer mentioned this pull request Jul 9, 2021
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.

4 participants