Skip to content

Comments

some methods needed in hermes#226

Merged
Siel merged 2 commits intooutputs-outputs-outputsfrom
outputsx4
Nov 17, 2025
Merged

some methods needed in hermes#226
Siel merged 2 commits intooutputs-outputs-outputsfrom
outputsx4

Conversation

@Siel
Copy link
Member

@Siel Siel commented Nov 17, 2025

No description provided.

Copy link
Collaborator

@mhovd mhovd left a comment

Choose a reason for hiding this comment

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

LGTM

@Siel Siel merged commit 0703111 into outputs-outputs-outputs Nov 17, 2025
1 check passed
@Siel Siel deleted the outputsx4 branch November 17, 2025 13:09
@github-actions
Copy link
Contributor

🐰 Bencher Report

Branchoutputsx4
Testbedrust-moan

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results
BenchmarkLatencyseconds (s)
bimodal_ke_npag📈 view plot
⚠️ NO THRESHOLD
24.79 s
bimodal_ke_npod📈 view plot
⚠️ NO THRESHOLD
10.39 s
bimodal_ke_postprob📈 view plot
⚠️ NO THRESHOLD
3.45 s
🐰 View full continuous benchmarking report in Bencher

Siel added a commit that referenced this pull request Nov 17, 2025
* Add method to write Theta with names and probs

* Add Posterior to NPResult

* Add predictions to NPResult

* Update predictions.rs

* Implement suggestions

* chore: Clean up some comments

* feat: some methods needed in hermes (#226)

* some methods needed in hermes


---------

Co-authored-by: Julian Otalvaro <juliandavid347@gmail.com>
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