Skip to content

Conversation

@ChrisRackauckas-Claude
Copy link
Contributor

Summary

  • Add JET.jl as a test dependency for static analysis
  • Create test/jet_test.jl with type stability checks for key functions
  • Include JET tests in the Core1 test group

Changes

test/jet_test.jl (new file)

Tests type stability of key hot-path functions using JET.report_opt:

  • accept_step! - main stepping function
  • setup_next_step! - RSWM algorithm setup
  • calculate_step! - noise generation
  • WHITE_NOISE_DIST - white noise distribution
  • wiener_randn - random number generation (scalar and array variants)

Project.toml

  • Add JET to [extras] section as test dependency
  • Add JET compat entry (0.8, 0.9)
  • Include JET in test targets

test/runtests.jl

  • Include jet_test.jl in the Core1 test group

Testing

  • JET analysis shows 0 issues for the entire package
  • All 6 optimization tests pass
  • Tests gracefully skip when JET is not available

Notes

JET.report_package was intentionally avoided in the test suite because it creates a virtualized module context that can interfere with subsequent tests. The JET.report_opt approach provides targeted type stability verification without this side effect.

Verification

using JET
using DiffEqNoiseProcess

# Full package analysis
result = JET.report_package("DiffEqNoiseProcess"; target_modules=(DiffEqNoiseProcess,))
length(JET.get_reports(result))  # Returns 0

cc @ChrisRackauckas


🤖 Generated with Claude Code

- Add JET.jl as a test dependency
- Create test/jet_test.jl with type stability checks for key functions:
  - accept_step!, setup_next_step!, calculate_step!
  - WHITE_NOISE_DIST
  - wiener_randn (scalar and array variants)
- Include jet_test.jl in the Core1 test group

JET.report_package analysis confirms no type instabilities in the package.
The new tests use JET.report_opt to verify type stability of hot paths
is maintained as the package evolves.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
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