Skip to content

VST Chugin - #47

Open
DBraun wants to merge 5 commits into
ccrma:mainfrom
DBraun:feature/vst
Open

VST Chugin#47
DBraun wants to merge 5 commits into
ccrma:mainfrom
DBraun:feature/vst

Conversation

@DBraun

@DBraun DBraun commented Jan 11, 2021

Copy link
Copy Markdown
Member

I've added a VST chugin.

Some notes:

  • Full instructions are in VST/README.md
  • It's only tested on Windows
  • It requires a git patch in the root directory. The patch is necessary in order to have a way to print the name of a VST parameter (vst.getParameterName(0)). I'm working on that here How to return string in a chugin function? #46

Comment thread vst_patch.patch
public:
// add reference (ge: april 2013: made these virtual)
- virtual void add_ref();
+ virtual void add_ref() {};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you submit a pull request against https://github.com/ccrma/chuck so that this patch isn't necessary here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes but I'm reluctant to do that because I don't know if it would have unwanted side effects.

Comment thread VST/tests/vst_effect_test.ck Outdated
@@ -0,0 +1,24 @@
"C:/VSTPlugIns/ValhallaRoom_x64.dll" => string vst_effect_path; // Must be absolute path.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are there MIT-licensed VST effect(s) we could use here instead of proprietary ones?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

What do you think of the 64-bit Dimension Expander plugin here? https://xferrecords.com/freeware/ It's labeled freeware and is working with my chuck test. Would it be ok to add the DLL to the repo?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've added the DLL in this commit 79c3e59

Comment thread VST/tests/vst_instrument_test.ck Outdated
@@ -0,0 +1,28 @@
"C:/VSTPlugIns/Serum_x64.dll" => string vst_path; // Must use absolute path.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are there MIT-licensed VST instrument(s) we could use here instead of proprietary ones?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

What do you think of Tal-NoiseMaker? It's GPL and working with my test. https://tal-software.com/products/tal-noisemaker

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've add the DLL in this commit 79c3e59

@DBraun DBraun mentioned this pull request May 5, 2021
@marisevzyl marisevzyl changed the title Feature/vst Merge David's VST chugin Apr 9, 2023
@marisevzyl marisevzyl changed the title Merge David's VST chugin VST Chugin Apr 9, 2023
@spencersalazar

Copy link
Copy Markdown
Member

@DBraun sorry for the extreme delay, this looks super cool (I definitely want the same thing) but would recommend

  • using JUCE as a submodule instead of copying the code
  • ideally figure out x-platform support (chuck dev community can potentially help w this)
  • (stretch goal) generalize this to any JUCE-support plugin type (e.g. AU on Windows,
  • setting this up as a separate repo that people can use to access this chugin at least until all of the above are satisfied

To the last point -- there is some interest in having the ccrma/chugins repo be cross-platform and validated as generally working well for people, but also supporting community-developed chugins that don't meet all of these criteria (yet). For the latter, there is some discussion of a chuck/chugin package manager that could facilitate this.

@DBraun

DBraun commented Apr 25, 2023

Copy link
Copy Markdown
Member Author

Agreed. I think having a projucer file is good for maintenance. This is how DawDreamer works, and it supports macOS universal builds, which I know is the main cross platform headache. I may get around to a fresh VST chugin in a few months… If anyone wants to beat me to it (please), let us know here. I can offer some guidance through DawDreamer.

@nshaheed

Copy link
Copy Markdown
Contributor

So, I have not looked into this extensively, but @dbadb has a vst3 chugin on their fork, perhaps this solves some of the issues (I don't think it does, but it's good to know what's out there)

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