-
Notifications
You must be signed in to change notification settings - Fork 9
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
[TTFX] Use Packages Extensions to drop Zygote and others when not used. #62
Comments
Taking a second look at this, It appear to me that it is only the last two files
Tell me what you think |
|
For SliceMap: Thanks for removing it ! For Zygote : by weak-dep you mean package extension, right ? |
Zygote has been added as an extension in c11dd72 |
TaylorDiff is depending on a lot of stuff it should not, which tanks my TTFX.
As already noted in #52 (comment)_ , It would be very beneficial to leverage package preferences to avoid, say, having Zygote and ReverseDiff as dependencies. Formally, the composability with the AD systems is not a core functionality of the package and could be loaded ONLY when those AD systems are loaded.
The text was updated successfully, but these errors were encountered: