Skip to content

refactor: remove StaticArrays dependency and optimize polarization computations#17

Merged
Beforerr merged 1 commit into
mainfrom
push-tukuokwwkksr
May 10, 2026
Merged

refactor: remove StaticArrays dependency and optimize polarization computations#17
Beforerr merged 1 commit into
mainfrom
push-tukuokwwkksr

Conversation

@Beforerr
Copy link
Copy Markdown
Member

  • Implement manual 3×3 Hermitian eigendecomposition for SVD polarization

@codecov
Copy link
Copy Markdown

codecov Bot commented May 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 10, 2026

Benchmark Results (Julia v1)

Time benchmarks
main 374de80... main / 374de80...
spectral_matrix/complex 2.53 ± 0.27 μs 2.52 ± 0.22 μs 1.01 ± 0.14
spectral_matrix/real 0.0982 ± 0.0078 ms 0.0975 ± 0.0074 ms 1.01 ± 0.11
wavpol/eigen 2.87 ± 0.097 ms 2.88 ± 0.071 ms 0.997 ± 0.042
wavpol/svd 1.74 ± 0.036 ms 1.75 ± 0.04 ms 0.994 ± 0.031
time_to_load 0.178 ± 0.0022 s 0.176 ± 0.0017 s 1.01 ± 0.016
Memory benchmarks
main 374de80... main / 374de80...
spectral_matrix/complex 3 allocs: 18.3 kB 3 allocs: 18.3 kB 1
spectral_matrix/real 11 allocs: 0.376 MB 11 allocs: 0.376 MB 1
wavpol/eigen 0.048 k allocs: 0.225 MB 0.048 k allocs: 0.225 MB 1
wavpol/svd 0.047 k allocs: 0.195 MB 0.047 k allocs: 0.195 MB 1
time_to_load 0.145 k allocs: 11 kB 0.145 k allocs: 11 kB 1

…mputations

- Implement manual 3×3 Hermitian eigendecomposition for SVD polarization
@Beforerr Beforerr force-pushed the push-tukuokwwkksr branch from 77b7ce7 to 374de80 Compare May 10, 2026 19:38
@Beforerr Beforerr merged commit 98a0f7d into main May 10, 2026
4 checks passed
@Beforerr Beforerr deleted the push-tukuokwwkksr branch May 10, 2026 19:38
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