-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Release patch with runtime activity support? #128
Comments
Note that for runtime activity to be supported here it might have to be supported in DI, which requires a merge on EnzymeAD/Enzyme.jl#1979. |
It does not and it is already implemented here.
OptimizationBase does not use DI for Enzyme
…On Tue, Nov 5, 2024 at 2:34 PM Guillaume Dalle ***@***.***> wrote:
Note that for runtime activity to be supported here it might have to be
supported in DI, which requires a merge on EnzymeAD/Enzyme.jl#1979
<EnzymeAD/Enzyme.jl#1979>.
I think that PR is good to go anyway?
—
Reply to this email directly, view it on GitHub
<#128 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJTUXAGXEOT5B45Y7MNGSTZ7ETUDAVCNFSM6AAAAABRHLIC6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGEYDEMJZHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Except for sparse AD? |
I am specifically asking to make a patch release of this landed PR #124 , since people are hitting issues otherwise. Nothing here depends on DI |
And I was just using the issue to highlight that people will hit similar issues if they use Enzyme in a sparse setting with OptimizationBase, unless a very short and already approved PR is merged to Enzyme 😇 |
Sorry for the disturbance |
I bumped |
It landed 2 weeks ago but look like it hasn't had a release.
Seems to be blocking some downstream things
The text was updated successfully, but these errors were encountered: