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

Make foo-yc20 compilable against faust 2.20 #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

audiofranky
Copy link

Hi Sampo,

long no hear :-). After an OS reinstall (LinuxMint 20) at home, I noticed that I couldn't build foo-yc20 anymore due to Faust updates.

With these changes, it builds&works for me:

  • Regenerated ("make generate-source") sources with faust 2.20.2
  • Fixed include statements for dsp.h
  • Implemented empty function body for addSoundfile()

Greetings,
Frank

* Regenerated ("make generate-source") sources with faust 2.20.2
* Fixed include statements for dsp.h
* Implemented empty function body for addSoundfile()
@sampov2
Copy link
Owner

sampov2 commented Feb 9, 2021

Sounds great Frank! There have been some performance issues with later versions of Faust, can you comment on whether the resulting code is running smoothly and not using up all CPU capacity?

@audiofranky
Copy link
Author

Hmm..I wanted to compare this build to the package that can be installed from the official sources (Ubuntu Focal Fossa, holds foo-yc20 version 1.3.0-6build3). However, that old package - while perfectly installable - is not running in my host (Qtractor) anymore; "YC20: Host incapable of running..".
Anyway, on my aging PC (Intel 2600K, from 2012), I see a CPU load of roughly 4-6% when the plugin is active in Qtractor (and that doesn't change when I put my both arms on as many keys on the MID keyboard as I can - but foo-yc20 is fully polyphonic all the time anyway, right?). Didn't try on my even older Netbook though in case you remember that :-).
So, I'd carefully say this works ok and at reasonable CPU load here.

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.

None yet

2 participants