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

test: re-enable flux testing #1123

Merged
merged 2 commits into from
Dec 6, 2024
Merged

test: re-enable flux testing #1123

merged 2 commits into from
Dec 6, 2024

Conversation

avik-pal
Copy link
Member

@avik-pal avik-pal commented Dec 5, 2024

fixes #1070

This pull request includes several changes to the LuxFluxExt.jl and test files to re-enable previously disabled functionality and update test configurations. The most important changes are grouped by theme below:

Test Configuration:

  • Re-enabled the addition of the Flux package in the EXTRA_PKGS list in test/runtests.jl.
  • Removed the skip condition from the FromFluxAdaptor test item in test/transform/flux_tests.jl, allowing the test to run.

Copy link
Contributor

github-actions bot commented Dec 5, 2024

Benchmark Results (ASV)

main 858ef12... main/858ef126428b7c...
basics/overhead 0.142 ± 0.0019 μs 0.131 ± 0.0031 μs 1.09
time_to_load 1.3 ± 0.0071 s 1.3 ± 0.0059 s 0.998

Benchmark Plots

A plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR.
Go to "Actions"->"Benchmark a pull request"->[the most recent run]->"Artifacts" (at the bottom).

@avik-pal avik-pal merged commit fb44a48 into main Dec 6, 2024
32 of 48 checks passed
@avik-pal avik-pal deleted the ap/flux_testing branch December 6, 2024 05:17
@Vaibhavdixit02
Copy link

@avik-pal we need a release with this branch to successfully test Optimisers v0.4 in SciML/Optimization.jl#861 - this is holding back Flux which is holding back Optimisers in the resolver there

@avik-pal
Copy link
Member Author

avik-pal commented Dec 7, 2024

@avik-pal we need a release with this branch to successfully test Optimisers v0.4 in SciML/Optimization.jl#861 - this is holding back Flux which is holding back Optimisers in the resolver there

done

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.

Re-enable Flux compatibility testing
2 participants