-
-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
export ps_khat_threshold #358
Comments
Agreed. Would you mind making a small PR for this? |
Discussing this with @avehtari we thought that it might be sensible to export these kinds of functions under a new documentation heading "helpers", which would be for functions to be used by other packages and not (likely) used interactively. Other functions include those listed in stan-dev/loo#249 |
okay by me
n-kall ***@***.***> schrieb am Fr., 22. März 2024, 16:22:
… Discussing this with @avehtari <https://github.com/avehtari> we thought
that it might be sensible to export these kinds of functions under a new
documentation heading "helpers", which would be for functions to be used by
other packages and not (likely) used interactively. Other functions include
those listed in stan-dev/loo#249
<stan-dev/loo#249>
—
Reply to this email directly, view it on GitHub
<#358 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADCW2AHQYGC6GT73RP3FN6LYZQ5ARAVCNFSM6AAAAABFC3VZCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGIYTGMJVGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Merged
Closed
This can be closed, now that #361 is merged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
loo
,brms
,projpred
, and eventuallybayesplot
would benefit fromps_khat_threshold()
being an exported function. Currentlyloo
andbrms
copy that, andprojpred
is going to copy that.The text was updated successfully, but these errors were encountered: