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

reimplemented on latest upstream (August 11th, 2024) #162

Merged
merged 10 commits into from
Aug 15, 2024
Merged

Conversation

ZuseZ4
Copy link
Member

@ZuseZ4 ZuseZ4 commented Aug 11, 2024

Hopefully a base to discuss upstreaming :)

@ZuseZ4 ZuseZ4 mentioned this pull request Aug 12, 2024
@jedbrown
Copy link

My test with rsmpi passes with this (bumping the Enzyme commit).

What happened to the EnzymeAD/rustbook test suite? I see no checks in this PR nor references to that repo, but I don't want to lose that (even if it will have to be done differently to go upstream).

@ZuseZ4
Copy link
Member Author

ZuseZ4 commented Aug 15, 2024

Awesome. I'll bump Enzyme, clean this up a bit and make upstream PRs.
No CI (yet), since I will leave upstreaming tests for slightly later to simplify the upstreaming, so I run them locally in the meantime. In a few days I'll add them back.

@jedbrown
Copy link

Can you post the manual procedure you're using somewhere so it doesn't get lost?

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