Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Add JSON API. #11

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

Add JSON API. #11

wants to merge 1 commit into from

Conversation

jonashaag
Copy link
Contributor

Also optimise Dockerfile for quick rebuilds.

Also add multiple input files support to predict.py.

Is it expected that the estimate for 5 speakers example is 6?

@jonashaag
Copy link
Contributor Author

Also, what is the purpose of predict_audio.py?

@faroit
Copy link
Owner

faroit commented Sep 4, 2020

@jonashaag as far as I remember I got 6 with the tf backend and 5 with theano. But I will check later.

@faroit
Copy link
Owner

faroit commented Sep 4, 2020

@jonashaag can be removed

@faroit
Copy link
Owner

faroit commented Sep 4, 2020

Is it expected that the estimate for 5 speakers example is 6?

i checked again, for both CRNN and RNN i get the correct response (5). So you probably have fallen into the wrong tf backend.

Also optimise Dockerfile for quick rebuilds.

Also add multiple input files support to `predict.py`.
@jonashaag
Copy link
Contributor Author

OK, changed to Theano backend. I also switched to Conda so that any numeric libraries (such as MKL) are automatically installed

@faroit
Copy link
Owner

faroit commented Sep 5, 2020

Okay. Great. I will probably add a simple unit test to make sure that the backends are correct

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

Successfully merging this pull request may close these issues.

2 participants