Skip to content

fix: Normalize psi to prevent likelihood underflow - #300

Merged
Siel merged 2 commits into
mainfrom
fix/0.26-psi-log-scale
Jul 20, 2026
Merged

fix: Normalize psi to prevent likelihood underflow#300
Siel merged 2 commits into
mainfrom
fix/0.26-psi-log-scale

Conversation

@Siel

@Siel Siel commented Jul 20, 2026

Copy link
Copy Markdown
Member

Summary

  • scale each subject likelihood row before exponentiation
  • restore the scale in IPM and NCNPAG objectives
  • add regression tests

Tests

  • cargo test --lib
  • cargo fmt --check
  • cargo check

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

ProjectPMcore
Branchfix/0.26-psi-log-scale
Testbedmhovd-pgx
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
bimodal_ke_npag📈 view plot
🚷 view threshold
3,859.80 ms
(-21.33%)Baseline: 4,906.32 ms
6,113.39 ms
(63.14%)
bimodal_ke_npod📈 view plot
🚷 view threshold
1,433.50 ms
(-0.49%)Baseline: 1,440.50 ms
2,121.87 ms
(67.56%)
bimodal_ke_postprob📈 view plot
🚷 view threshold
522.81 ms
(+43.38%)Baseline: 364.64 ms
926.99 ms
(56.40%)
🐰 View full continuous benchmarking report in Bencher

@mhovd mhovd changed the title Fix likelihood underflow fix: Normaliz psi to prevent likelihood underflow Jul 20, 2026
@mhovd mhovd changed the title fix: Normaliz psi to prevent likelihood underflow fix: Normalize psi to prevent likelihood underflow Jul 20, 2026

@mhovd mhovd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful

@Siel
Siel merged commit 588de92 into main Jul 20, 2026
7 checks passed
@mhovd
mhovd deleted the fix/0.26-psi-log-scale branch July 20, 2026 19:47
@github-actions github-actions Bot mentioned this pull request Jul 20, 2026
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