Skip to content

Exclude Enzyme from pre-release tests #644

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

Merged
merged 12 commits into from
Jul 19, 2025
Merged

Exclude Enzyme from pre-release tests #644

merged 12 commits into from
Jul 19, 2025

Conversation

ChrisRackauckas
Copy link
Member

No description provided.

@@ -9,11 +9,23 @@ end
using LineSearches: LineSearches
using BenchmarkTools: @ballocated
using StaticArrays: @SVector
using Zygote, Enzyme, ForwardDiff, FiniteDiff
using Zygote, ForwardDiff, FiniteDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@@ -9,11 +9,23 @@ end
using LineSearches: LineSearches
using BenchmarkTools: @ballocated
using StaticArrays: @SVector
using Zygote, Enzyme, ForwardDiff, FiniteDiff
using Zygote, ForwardDiff, FiniteDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using SciMLJacobianOperators

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@@ -1,6 +1,12 @@
@testsnippet RootfindTestSnippet begin
using StaticArrays, Random, LinearAlgebra, ForwardDiff, NonlinearSolveBase, SciMLBase
using ADTypes, PolyesterForwardDiff, Enzyme, ReverseDiff
using ADTypes, PolyesterForwardDiff, ReverseDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using Zygote, ForwardDiff, FiniteDiff

# Conditionally import Enzyme only if not on Julia prerelease
if isempty(VERSION.prerelease)
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if isempty(VERSION.prerelease)
if isempty(VERSION.prerelease)

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@@ -93,14 +104,25 @@
using ADTypes, Random, LinearSolve, LinearAlgebra
using BenchmarkTools: @ballocated
using StaticArrays: @SVector
using Zygote, Enzyme, ForwardDiff, FiniteDiff
using Zygote, ForwardDiff, FiniteDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using Zygote, ForwardDiff, FiniteDiff

# Conditionally import Enzyme only if not on Julia prerelease
if isempty(VERSION.prerelease)
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if isempty(VERSION.prerelease)
if isempty(VERSION.prerelease)

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using SciMLJacobianOperators

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using SciMLJacobianOperators

# Conditionally import Enzyme only if not on Julia prerelease
if isempty(VERSION.prerelease)
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if isempty(VERSION.prerelease)
if isempty(VERSION.prerelease)

using SciMLJacobianOperators

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

using SciMLJacobianOperators

# Conditionally import Enzyme only if not on Julia prerelease
if isempty(VERSION.prerelease)
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
if isempty(VERSION.prerelease)
if isempty(VERSION.prerelease)

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@@ -84,9 +95,21 @@
using LineSearches: LineSearches
using BenchmarkTools: @ballocated
using StaticArrays: @SVector
using Zygote, Enzyme, ForwardDiff, FiniteDiff
using Zygote, ForwardDiff, FiniteDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

if isempty(VERSION.prerelease)
push!(autodiff_backends, (AutoEnzyme(), "no jac - enzyme"))
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

if isempty(VERSION.prerelease)
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@@ -157,9 +179,21 @@
using LineSearches: LineSearches
using BenchmarkTools: @ballocated
using StaticArrays: @SVector
using Zygote, Enzyme, ForwardDiff, FiniteDiff
using Zygote, ForwardDiff, FiniteDiff

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

if !is_julia_prerelease()
push!(autodiff_backends, AutoEnzyme())
end

Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

@ChrisRackauckas ChrisRackauckas merged commit 69f10db into master Jul 19, 2025
70 of 93 checks passed
@ChrisRackauckas ChrisRackauckas deleted the pre_enzyme branch July 19, 2025 22:17
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.

1 participant