Skip to content

Optimize particle_waves_v6 hot-path allocations#47

Open
mochell wants to merge 1 commit into
mainfrom
pr/particle-waves-v6-opt
Open

Optimize particle_waves_v6 hot-path allocations#47
mochell wants to merge 1 commit into
mainfrom
pr/particle-waves-v6-opt

Conversation

@mochell

@mochell mochell commented Jun 1, 2026

Copy link
Copy Markdown
Owner

Summary

This PR isolates allocation/performance-focused changes in particle_waves_v6.jl into a standalone review.

Changes

  • Avoid per-call closure allocation for propagation correction fallback.
  • Reuse αₚ intermediate instead of recomputing.
  • Replace scalar broadcast expressions with scalar arithmetic.
  • Remove temporary vector/slice allocations in propagation update by writing dz[4] and dz[5] directly.

Motivation

Keeps optimization-specific changes separate from the ongoing forcing-structure refactor branch, so each can be reviewed and merged independently.

Base automatically changed from change_forcing_handling to main June 4, 2026 14:34
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