Skip to content

reduce default popsize parameter from 170,000 to 50,000#178

Merged
justin-ven merged 1 commit intodevelopfrom
bugfix/reduce-default-gui-pop-size
Jun 10, 2025
Merged

reduce default popsize parameter from 170,000 to 50,000#178
justin-ven merged 1 commit intodevelopfrom
bugfix/reduce-default-gui-pop-size

Conversation

@andrewbaxter439
Copy link
Copy Markdown
Collaborator

Minor tweak - reduces default population size to a more manageable 50,000 in the GUI parameters. Very unlikely to change much in almost any runs but perhaps a useful tweak to avoid things hanging if a new user wants to attempt a run.

@andrewbaxter439 andrewbaxter439 requested a review from pbronka May 28, 2025 12:36
@pbronka
Copy link
Copy Markdown
Contributor

pbronka commented May 28, 2025

@justin-ven I think you were recommending at least a 170k - what's your take? I'm happy with reducing this to 50k by default; we would still use a larger population for any "meaningful" runs.

@andrewbaxter439
Copy link
Copy Markdown
Collaborator Author

Thanks @pbronka - I'd made this suggestion under the assumption that I had perhaps mistakenly put it up so high! If it's by design then perhaps ignore this PR. The 50,000-person setting would be overridden in all multiruns of interest anyway, but perhaps provides an easy 'sandbox mode' for new users to experiment in the GUI?

Very, very low priority in either case, so would suggest closing if not convincing.

@andrewbaxter439 andrewbaxter439 added the enhancement New feature or request label May 30, 2025
@justin-ven
Copy link
Copy Markdown
Contributor

The 170,000 starting population was broadly needed to generate sufficient observations for the care analysis. 50,000 ought to be fine for studies relating to the population in general, but it would be useful to bear in mind that this should probably be considered a "minimum" base for analysis.

@justin-ven justin-ven merged commit 1e6d59d into develop Jun 10, 2025
4 checks passed
@andrewbaxter439 andrewbaxter439 deleted the bugfix/reduce-default-gui-pop-size branch June 10, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants