Skip to content
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

Don't use fix1 for enzyme #4

Merged
merged 3 commits into from
Dec 24, 2022
Merged

Don't use fix1 for enzyme #4

merged 3 commits into from
Dec 24, 2022

Conversation

wsmoses
Copy link
Contributor

@wsmoses wsmoses commented Dec 24, 2022

No description provided.

@devmotion
Copy link
Collaborator

Thanks, that's much better 🙂 Does this also fix an open issue? Maybe even the Turing-related one?

@wsmoses
Copy link
Contributor Author

wsmoses commented Dec 24, 2022

Not yet, am trying to minimize that presently.

src/AD_Enzyme.jl Outdated Show resolved Hide resolved
src/AD_Enzyme.jl Outdated Show resolved Hide resolved
wsmoses and others added 2 commits December 24, 2022 16:51
Co-authored-by: David Widmann <[email protected]>
Co-authored-by: David Widmann <[email protected]>
@devmotion
Copy link
Collaborator

Not yet, am trying to minimize that presently.

BTW I saw your comment in the issue in the Enzyme repo but unfortunately I haven't been able to debug and minimize it further yet, I only have very limited access to my computer during the Christmas break.

@devmotion devmotion merged commit da73c64 into tpapp:master Dec 24, 2022
@devmotion devmotion mentioned this pull request May 12, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants