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

Has anyone built this and tested the demo in Windows> #99

Open
sumanrbt opened this issue Dec 2, 2019 · 7 comments
Open

Has anyone built this and tested the demo in Windows> #99

sumanrbt opened this issue Dec 2, 2019 · 7 comments
Labels

Comments

@sumanrbt
Copy link

sumanrbt commented Dec 2, 2019

No description provided.

@sumanrbt
Copy link
Author

sumanrbt commented Dec 2, 2019

I cloned the repo and built it using WSL ubuntu and the demo is working, is there a way to get a .exe of this project?

@dm8tbr dm8tbr added the question label Dec 2, 2019
@BenedicteLeo
Copy link

I did build an exe in Windows. However, when I run it it only produces garbage.

To build the demo, install Mingw and add its binary path to the path environment variables. I think you also need to add the msys binary path that comes with Mingw. Then you can run autogen, configure and make.

The input I pass in is a raw pcm 48kHz 16bit little endian file of size ~1000kB. The output I get is 40kB, lasts way shorter and sounds like a high pitch scratch. I've tried every possible input possible (different endianness, etc.) and nothing works. Please let me know if it works for you.

@BenedicteLeo
Copy link

I just found this in the resolved issues: https://github.com/jagger2048/rnnoise-windows. I'm about to give it a try!

@BenedicteLeo
Copy link

The VS project works for me. I have no idea why the default example doesn't work.

@LittleKita
Copy link

It also run on cygwin. You can build and run the rnnoise_demo.exe, just as if you were doing it under linux. (./autogen.sh ./configure make)
If you want the run the exe outside of cygwin you need the cygwin1.dll in the same directory.
For compiling it is better to use the cygwin-gcc not the cygwin-minggw-gcc.

@wegylexy
Copy link
Contributor

You need to read and write files in binary mode (rb and wb) on Windows, because \r and \n are handled differently in text mode (r and w).

@petterreinholdtsen
Copy link
Contributor

I guess #88 indicate that someone is working on a Windows edition. It does not adjust the file opening mode, but perhaps it should.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants