-
-
Notifications
You must be signed in to change notification settings - Fork 16.9k
Simplify autodiff handling of dependencies (rlib) #149164
Copy link
Copy link
Labels
E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.F-autodiff`#![feature(autodiff)]``#![feature(autodiff)]`
Description
Activity
Metadata
Metadata
Assignees
Labels
E-easyCall for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.F-autodiff`#![feature(autodiff)]``#![feature(autodiff)]`
We recently landed autodiff support for rlib's.
That means, you can now use
#[autodiff...]in a lib.rs file or even a dependency of your crate, use it in your main.rs file, and it still works. While discussing that last PR it was noticed that autodiff is likely a bit more complex than it has to be, so it would be good to delete the extra code.#149033 has most of the details and introduces two FIXME's.
Especially the thread around this comment lays out how we should simplify it. To quote bjorn3: