ch-nry / VCV_nozori

port of nozori eurorack module for VCV Rack
GNU General Public License v3.0
15 stars 2 forks source link

Migration to V2 #18

Closed netboy3 closed 2 years ago

netboy3 commented 2 years ago

I've just completed migrating the plugin to V2 and making a few enhancements. I've added a dark theme in the spirit of following the actual hardware modules. I've posted a PR #17 for your review and merge. I'm attaching below pre-built test binaries of the plugin for all three platforms. Please make sure to open the zip file below and pick the plugin binary for your platform. The binaries were built from modified sources posted to my repository and built as of commit da9fc03.

Nozoid-2.1.3.zip

ch-nry commented 2 years ago

wow, thanks for your work. After a quick look, I found that the "bad sample rate" warning is not updated when the sample rate change. (it appear or not depending on the SR when the module is created, but did not disappear (or appear) when the SR is changed). Anyway, I'll merge as soon as I have a bit more time. thanks!

netboy3 commented 2 years ago

You are very welcome! What you describe is actually not the module's code issue but rather the way Rack behaves. I noticed it too and put a debugger on the code. Rack does not change the value of args.sampleRate unless there's an active audio interface. I don't know if that's how Rack1 behaved, but that's how Rack2 works. It's actually logical as usually for most digital audio systems, clock and sync are derived from the digital audio clock source. Try it out - just drop an audio-8 or audio-16 module into the patch (no need to even connect it), make sure to configure it to a proper interface so it's active, and the warnings will work fine on SampleRate changes.

ch-nry commented 2 years ago

oh, yes, you are right!

netboy3 commented 2 years ago

Thanks for merging. I hope you'll be able to find a few minutes to request an update in the VCV library. Have a happy new year.

ch-nry commented 2 years ago

I just made the request. Happy new year, and thanks again!