-
Notifications
You must be signed in to change notification settings - Fork 188
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
How come the compiled version has more features than the source code? #22
Comments
Yeah, the compiled version is a C++ port of the python one with just a few small differences like the instrument button. I usually don't release pre-trained models for a few reasons:
Hopefully it should not be too hard to train a similar model yourself with the code provided. |
Thanks for the explanation, sounds all reasonable!
…On Thu, May 9, 2019, 23:47 HackerPoet ***@***.***> wrote:
Yeah, the compiled version is a C++ port of the python one with just a few
small differences like the instrument button.
I usually don't release pre-trained models for a few reasons:
- My C++ framework is custom-built and the models are not really
compatible with existing tools
- I don't really have time to document the data format and usage
- The original models are really large (in this case 350MBs) and I
don't have a good place to host it
- There could be privacy or ownership issues depending on the data used
- I don't want the pre-trained models to be used commercially (code is
MIT licenced though)
Hopefully it should not be too hard to train a similar model yourself with
the code provided.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAXXXKZU4Z37ABCHDSPGBEDPUSLXFANCNFSM4HLVNCQQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I could at least spot that the compiled version has the instrument buttons. Also, would you mind releasing the model?
The text was updated successfully, but these errors were encountered: