-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Native dependencies of ML.NET #895
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
Comments
Do you know how to eliminate the import of |
Here's what we do for CLRCompression. Looks like we statically link it. Some similar options defined in CoreCLR: |
We’ve started static linking the VC Runtime in #1828. Closing. |
We should review and try to unify and document the native dependencies of ML.NET, in particular the VC-runtime.
Ideally we should try and build the native components in the same way that .NET Core does so that we don't impose any new requirements (eg: installing a different / specific version of the VC redist).
I checked a few DLLs using link /dump /imports and noticed a lot of instances of VCRUNTIME140 which we do not depend on for coreclr/clrcompression/clrjit/etc.
I just did this check for Windows but we should also look at a similar thing on linux.
We should also examine binary redists (like TensorFlow) so that we can understand if they are different and at least document their dependencies.
/cc @shauheen @tannergooding @danmosemsft @eerhardt
The text was updated successfully, but these errors were encountered: