-
Notifications
You must be signed in to change notification settings - Fork 17
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
cvOSCcv development version can crash Rack 2 #60
Comments
Hi. Sorry for the late reply. Thanks for the report. That is definitely not good... We have tried to replicate this deleting both the sending (left) and the receiving (right) cvOSCcv modules and have not had it crash yet. It doesn't crash with the default send rate at 100 Hz or at 1000 Hz either. I tried on the free version of VCV Rack (just using Fundamental modules with it). j4s0n tried on the Pro version with all the same modules in the image except for the scope you used. What OS are you using? And do you have the save file for this patch? Are you able to consistently replicate this? Can you replicate it with just simple modules (like VCV Rack Fundamental, in case we don't have all the same modules)? If you could upload a save file where this happens for you, we can hopeful get it to crash on us too. |
I forgot to see your reply - sorry about that. On windows 11 22H2 (OS Build 22621.521) - using Rack Free 2.1.2 I can reproduce it: enable both cvOSCcv modules No error is logged in log.txt Thank you for adding the "Send Frequency / Hz" option. |
Thanks. I will try to look at it later tonight, but I'm working on a different project now. |
Issue #60. Fixed where deleting an active cvOSCcv receiver may crash rack.
Hi. This has been fixed in the latest release v2.0.6 and should update in the library in the next week or so. |
I have a bug report.
Deleting the cvOSCcv module on the right, when it's "Enabled", crashes Rack2 pro standalone.
Also, I changed TROWA_OSCCV_DEFAULT_SEND_HZ to 1000
Please make the default send freq a runtime configurable option.
Thank you for making these plugins available.
I use them frequently, and recommend them in the community forum:
https://community.vcvrack.com/t/ui-your-opinion-and-advice-appreciated/15713/6
https://community.vcvrack.com/t/thinking-about-purchasing-the-es-9/16228/10
https://community.vcvrack.com/t/bring-in-measurements-from-a-digital-multimeter/14557
The text was updated successfully, but these errors were encountered: