Skip to content

Warn when a seed is specified but adaptive stopping is still enabled - #90

Merged
bob-carpenter merged 3 commits into
mainfrom
python/warn-seed-but-nonreproducible
Aug 11, 2026
Merged

Warn when a seed is specified but adaptive stopping is still enabled#90
bob-carpenter merged 3 commits into
mainfrom
python/warn-seed-but-nonreproducible

Conversation

@WardBrian

Copy link
Copy Markdown
Collaborator

No description provided.

@WardBrian WardBrian closed this Aug 7, 2026
@WardBrian WardBrian reopened this Aug 7, 2026
@WardBrian
WardBrian requested a review from bob-carpenter August 7, 2026 17:20
Comment thread python/src/walnutpie/pyfunc.py Outdated
num_params = init_shape[0]

seed = prepare_seed(seed)
is_adaptive = num_chains > 1 and (

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.

is_adaptive is a confusing name here given there's multiple adaptations in play. Can we call this something like is_monitored?

And how often do we run if is_monitored is False, but min_warmup_iter != max_warup_iter? Do we just take the minimum?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

And how often do we run if is_monitored is False, but min_warmup_iter != max_warup_iter? Do we just take the minimum?

What do you mean by this?

Comment thread python/src/walnutpie/__init__.py
Comment thread python/src/walnutpie/stan.py Outdated
Comment thread python/src/walnutpie/util.py Outdated

@bob-carpenter bob-carpenter left a comment

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.

Thanks for factoring out an adaptivity function.

@bob-carpenter
bob-carpenter merged commit ab9895f into main Aug 11, 2026
19 checks passed
@WardBrian
WardBrian deleted the python/warn-seed-but-nonreproducible branch August 11, 2026 16:29
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