Skip to content

fix: Add wrappers for analytical solutions#272

Merged
mhovd merged 4 commits into
mainfrom
fix/pmetrics-analytical
Apr 20, 2026
Merged

fix: Add wrappers for analytical solutions#272
mhovd merged 4 commits into
mainfrom
fix/pmetrics-analytical

Conversation

@Siel
Copy link
Copy Markdown
Member

@Siel Siel commented Apr 20, 2026

No description provided.

@Siel Siel marked this pull request as ready for review April 20, 2026 17:18
Copilot AI review requested due to automatic review settings April 20, 2026 17:18
@mhovd mhovd changed the title Fix/pmetrics analytical fix: Add wrappers for analytical solutions Apr 20, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates dependency versions in pmcore to align with the intended analytical/Pmetrics-related fixes by adjusting crate requirements.

Changes:

  • Bump pharmsol from =0.26 to =0.26.1.
  • Change rand dependency from 0.10.1 to 0.9.0.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mhovd mhovd merged commit 88ce1c6 into main Apr 20, 2026
7 checks passed
@mhovd mhovd deleted the fix/pmetrics-analytical branch April 20, 2026 17:29
@github-actions github-actions Bot mentioned this pull request Apr 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 20, 2026

🐰 Bencher Report

Branchfix/pmetrics-analytical
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
4,576.80 ms
(-5.82%)Baseline: 4,859.71 ms
5,071.93 ms
(90.24%)
bimodal_ke_npod📈 view plot
🚷 view threshold
1,269.30 ms
(-7.46%)Baseline: 1,371.60 ms
1,630.27 ms
(77.86%)
bimodal_ke_postprob📈 view plot
🚷 view threshold
58.59 ms
(-80.14%)Baseline: 294.95 ms
762.05 ms
(7.69%)
🐰 View full continuous benchmarking report in Bencher

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.

3 participants