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

Model mismatch with basic-pitch-ts ? #62

Open
tiborvass opened this issue Feb 23, 2023 · 1 comment
Open

Model mismatch with basic-pitch-ts ? #62

tiborvass opened this issue Feb 23, 2023 · 1 comment

Comments

@tiborvass
Copy link

tiborvass commented Feb 23, 2023

Thanks for this great project! I would like to use this python project and get the same results as on https://basicpitch.spotify.com.

I am a newbie with TensorFlow but I'm assuming the models used in the basic-pitch and basic-pitch-ts projects are not exactly the same:

First of all: am I wrong in my assumptions ? If not, it would be great to have both projects use the same model (despite being in a different format).

I myself tried using both tensorflowjs_converter and tfjs_graph_converter but none worked or possibly I did something wrong.

Thanks for any help!

@tiborvass tiborvass changed the title Model mismatch with basic-pitch-ts Model mismatch with basic-pitch-ts ? Feb 23, 2023
@drubinstein
Copy link
Contributor

The model used in basic-pitch-ts is based on running the tfjs converter on the one in this repo. The test data is actually from running inference with the scripts here so they should be equivalent.

You may see some differences in the website based on the slider values used.

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

No branches or pull requests

2 participants