Skip to content

Re-activate GenrouEnzyme#293

Merged
pelesh merged 2 commits intodevelopfrom
nicholson/test-Genrou-locally
Oct 30, 2025
Merged

Re-activate GenrouEnzyme#293
pelesh merged 2 commits intodevelopfrom
nicholson/test-Genrou-locally

Conversation

@nkoukpaizan
Copy link
Collaborator

@nkoukpaizan nkoukpaizan commented Oct 28, 2025

Description

This re-activates GenrouEnzyme builds and related tests, as the underlying issue in Enzyme has been addressed.
Closes #234

Checklist

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • [N/A] The new code follows GridKit™ style guidelines.
  • [N/A] There are unit tests for the new code.
  • [N/A] The new code is documented.
  • The feature branch is rebased with respect to the target branch.
  • [N/A] I have updated CHANGELOG.md to reflect the changes in this PR. If this is a minor PR that is part of a larger fix already included in the file, state so.

Further comments

Waiting for the bug fix to be in an Enzyme release to merge.

@nkoukpaizan nkoukpaizan changed the base branch from develop to nicholson/external-var-Jacobian October 28, 2025 21:13
@nkoukpaizan nkoukpaizan force-pushed the nicholson/test-Genrou-locally branch from deee4a5 to 2d6fccf Compare October 28, 2025 21:15
@nkoukpaizan nkoukpaizan changed the title test genrou Re-activate GenrouEnzyme Oct 29, 2025
@pelesh
Copy link
Collaborator

pelesh commented Oct 29, 2025

@nkoukpaizan is this ready for review and (eventually) merge? This is still a draft PR.

@nkoukpaizan nkoukpaizan self-assigned this Oct 29, 2025
@nkoukpaizan nkoukpaizan added bug Something isn't working enzyme labels Oct 29, 2025
@nkoukpaizan nkoukpaizan marked this pull request as ready for review October 29, 2025 16:51
@nkoukpaizan
Copy link
Collaborator Author

Sure, it can be reviewed. I just need to remind myself to point to an Enzyme release.

@pelesh
Copy link
Collaborator

pelesh commented Oct 29, 2025

Perhaps we want to update GridKit spack and README now that @wsmoses merged sac branch to main Enzyme?

@nkoukpaizan nkoukpaizan force-pushed the nicholson/test-Genrou-locally branch from 34a5984 to 0d3dd89 Compare October 30, 2025 01:41
@nkoukpaizan nkoukpaizan changed the base branch from nicholson/external-var-Jacobian to develop October 30, 2025 01:41
Copy link
Collaborator

@pelesh pelesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ready to merge.

@nkoukpaizan nkoukpaizan force-pushed the nicholson/test-Genrou-locally branch from 0d3dd89 to d03bc25 Compare October 30, 2025 14:02
@pelesh pelesh merged commit 491ea31 into develop Oct 30, 2025
6 checks passed
@nkoukpaizan nkoukpaizan deleted the nicholson/test-Genrou-locally branch October 31, 2025 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enzyme

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build issue: GenrouEnzyme intermittently fails to build

3 participants