Skip to content

Remove ParameterizedFunctions.jl dependency from tests #2843

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

Conversation

ChrisRackauckas-Claude
Copy link

Summary

  • Removes the ParameterizedFunctions.jl dependency from OrdinaryDiffEq.jl tests
  • Replaces the @ode_def macro with standard Julia function definitions
  • Maintains compatibility with both in-place and out-of-place ODE problems

Changes

  1. Replaced the @ode_def Hires macro in test/interface/linear_solver_test.jl with:
    • hires! function for in-place operations
    • hires function for out-of-place operations
  2. Updated the functions to handle both Float64 and Float32 types properly
  3. Removed ParameterizedFunctions from Project.toml dependencies and test targets

Test plan

  • Verified the new functions work with FBDF solver
  • Tests pass for the Hires system with various solvers (QNDF, FBDF, Rodas5P)
  • Both Float64 and Float32 versions work correctly
  • Code formatted with JuliaFormatter using SciMLStyle

🤖 Generated with Claude Code

ChrisRackauckas and others added 2 commits August 13, 2025 07:05
Replace the @ode_def macro from ParameterizedFunctions with standard Julia
function definitions for both in-place and out-of-place versions of the
hires ODE system in linear_solver_test.jl.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas merged commit d58ffb1 into SciML:master Aug 13, 2025
161 of 195 checks passed
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