Skip to content

take out the sbm_prior from the bgmCompare documentation#98

Closed
sekulovskin wants to merge 0 commit into
Bayesian-Graphical-Modelling-Lab:mainfrom
sekulovskin:main
Closed

take out the sbm_prior from the bgmCompare documentation#98
sekulovskin wants to merge 0 commit into
Bayesian-Graphical-Modelling-Lab:mainfrom
sekulovskin:main

Conversation

@sekulovskin
Copy link
Copy Markdown
Contributor

@sekulovskin sekulovskin commented Apr 30, 2026

Description

The documentation for bgmCompare listed the sbm_prior as one of the options for the difference_prior argument. I assume this was generically written based on the documentation for bgm. Therefore I took it out from the documerntation.

You can also see that this should not be the case since running:

x = Boredom[Boredom$language == "fr", 2:6]
y = Boredom[Boredom$language != "fr", 2:6]

fit = bgmCompare(x, y, chains = 2, 
                 difference_prior = sbm_prior()
)

gives

Error in match.arg(difference_prior) : 
  'arg' should be one of “Bernoulli”, “Beta-Bernoulli”
Called from: match.arg(difference_prior)

@MaartenMarsman
Copy link
Copy Markdown
Collaborator

It was intended to work with sbm (it should be plug and play). Fixed it now. Can you verify?

@sekulovskin
Copy link
Copy Markdown
Contributor Author

It was intended to work with sbm (it should be plug and play). Fixed it now. Can you verify?

Confirmed!

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