Skip to content

Fix demes parser zeroing out sample sizes#80

Closed
nspope wants to merge 1 commit into
nsp-yaml-revampfrom
nsp-yaml-revap-patch-1
Closed

Fix demes parser zeroing out sample sizes#80
nspope wants to merge 1 commit into
nsp-yaml-revampfrom
nsp-yaml-revap-patch-1

Conversation

@nspope

@nspope nspope commented Apr 24, 2026

Copy link
Copy Markdown

Fixes #75.

  • Using a demes yaml from the CLI is now disabled. The -D flag is recognized but errors out. We should remove it in the future, and update the docs accordingly. Let's do this later once the YAML parser is finished.
  • The validation suite for demes now exits early with a message about the -D flag. These should be updated or removed at some later point. I'm not doing this now as there is other testing needed for the YAML parser (for parity with CLI) before re-testing the demes parser.

@nspope nspope changed the base branch from master to nsp-yaml-revamp April 24, 2026 18:09
@nspope

nspope commented Apr 24, 2026

Copy link
Copy Markdown
Author

Not sure why the testing has bottomed out here; going to close and open a new PR

@nspope nspope closed this Apr 24, 2026
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.

YAML demes_filename silently clobbers deme_sample_size, causing SIGSEGV

1 participant