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

AdlibBlasterx64 0.11.1 (AudioUnit) has digital noise upon start with DOSBox emulation only #34

Closed
douglas-carmichael opened this issue Jan 11, 2016 · 10 comments

Comments

@douglas-carmichael
Copy link

@douglas-carmichael douglas-carmichael commented Jan 11, 2016

System: Logic Pro X 10.2.0, AdlibBlasterx64 0.11.1.

When I start playing a project with the DOSBox emulation, there is a burst of digital noise (then the project starts playing). But, when I use the ZDoom emulation, there is no noise burst and the project plays as normal.
Attached is a ZIPed WAV file.

Adlib_DOSBox_DigiNoise.zip

@douglas-carmichael douglas-carmichael changed the title AdlibBlasterx64 0.11.1 has digital noise upon start with DOSBox emulation only AdlibBlasterx64 0.11.1 (AudioUnit) has digital noise upon start with DOSBox emulation only Jan 11, 2016
@bsutherland

This comment has been minimized.

Copy link
Owner

@bsutherland bsutherland commented Jan 11, 2016

Douglas, thanks for the detailed report. Unfortunately I don't have a Mac so I have no way to build or test on OSX :(

You could try reporting the issue to Jeff Russ who built the AudioUnit version.

https://github.com/Jeff-Russ/AdlibBlaster

Cheers.

@razvanab

This comment has been minimized.

Copy link

@razvanab razvanab commented Feb 6, 2016

The same thing happens but the noise persist until i switch to ZDoom in windows when i use this vst with vsthost from
http://www.hermannseib.com/english/vsthost.htm

@bsutherland

This comment has been minimized.

Copy link
Owner

@bsutherland bsutherland commented Feb 7, 2016

Thanks for the extra info razvanab.

@Jeff-Russ

This comment has been minimized.

Copy link
Contributor

@Jeff-Russ Jeff-Russ commented Feb 7, 2016

Everybody: I'm consumed in a web development project at the moment but I plan to return and fix the Mac build as soon as I can. Sorry I've been AWOL!!

@razvanab

This comment has been minimized.

Copy link

@razvanab razvanab commented Feb 7, 2016

I don't know if this is the right place to post this but i found out how to fix the noise in vsthost from:
http://www.hermannseib.com/english/vsthost.htm

In select Wave devices you need to set the buffer as low as possible.

@Jeff-Russ

This comment has been minimized.

Copy link
Contributor

@Jeff-Russ Jeff-Russ commented Feb 8, 2016

@douglas-carmichael @razvanab
This is by no stretch a "release" but I had some free time to I fired up Xcode. I just made a few changes and it compiled. It ran with no major issues in OSX 10.10 Logic Pro X and OSX 10.10 Ableton Live 9 as both AUi and VSTi. I didn't test it very much but see if this fixed whatever issue you had...

Download AUi
Download VSTi
my fork of JuceOPLVSTi

@bsutherland

This comment has been minimized.

Copy link
Owner

@bsutherland bsutherland commented Feb 9, 2016

Thanks razvanab! That might help me track it down.

I'll also put it in the wiki.

@jeremypenner

This comment has been minimized.

Copy link

@jeremypenner jeremypenner commented Mar 29, 2016

It looks like the problem is in DBOPL::Handler::Generate - if it's asked to generate more than 512 samples, it only puts 512 samples in the buffer, then stops. No idea what that logic is for, but if I comment it out, the noise appears to go away.

(Using GarageBand as an AU host for testing.)

I can put together a real pull request if that helps, but it seems to be a very simple fix.

@bsutherland

This comment has been minimized.

Copy link
Owner

@bsutherland bsutherland commented Mar 29, 2016

Jeremy, that sounds very promising! That would explain the issue at higher sample rates in some hosts. Thanks a lot for finding the issue and letting me know!!

I'll try to put out a new build within a week.

@bsutherland

This comment has been minimized.

Copy link
Owner

@bsutherland bsutherland commented May 14, 2016

This issue might be fixed in the latest Windows release. Closing this issue for now as there are many similar tickets.

https://github.com/bsutherland/JuceOPLVSTi/releases/tag/0.13.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.