Skip to content

Implement earnings-height model in pymc#269

Merged
JTorgander merged 7 commits intostan-dev:developmentfrom
jessegrabowski:earn-height-pymc
Feb 13, 2026
Merged

Implement earnings-height model in pymc#269
JTorgander merged 7 commits intostan-dev:developmentfrom
jessegrabowski:earn-height-pymc

Conversation

@jessegrabowski
Copy link
Copy Markdown

I followed the guide posted by @fonnesbeck here, let me know if this is missing anything!

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.

How has this file named been changed?

@JTorgander
Copy link
Copy Markdown
Collaborator

Thank you for contributing! Sorry for the delay. The model definition looks good to me and the model passes our consistency checks. I have modified the pymc code to take in data on the form which directly matches the posteriorDB JSON-data and removed some unnecessary extra files.

@JTorgander JTorgander changed the base branch from master to development January 23, 2026 08:38
@MansMeg
Copy link
Copy Markdown
Collaborator

MansMeg commented Feb 3, 2026

Dont we also need to add in the info file, @JTorgander ?
"pymc": {
"model_code": "models/pymc/...py",
"pymc_version": ">=5.16.2"

@MansMeg
Copy link
Copy Markdown
Collaborator

MansMeg commented Feb 9, 2026

LGTM feel free to merge @JTorgander

@JTorgander JTorgander merged commit 666c698 into stan-dev:development Feb 13, 2026
1 check passed
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