Skip to content

Fix spurious assertion in split_disjoint_nodes (issue #497)#509

Merged
nspope merged 1 commit intotskit-dev:mainfrom
nspope:split-disjoint-fix
Mar 20, 2026
Merged

Fix spurious assertion in split_disjoint_nodes (issue #497)#509
nspope merged 1 commit intotskit-dev:mainfrom
nspope:split-disjoint-fix

Conversation

@nspope
Copy link
Contributor

@nspope nspope commented Mar 17, 2026

No description provided.

@nspope nspope requested a review from hyanwong March 17, 2026 21:05
@nspope
Copy link
Contributor Author

nspope commented Mar 19, 2026

@hyanwong do you mind if I merge this (small but important) fix?

Copy link
Member

@hyanwong hyanwong left a comment

Choose a reason for hiding this comment

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

LGTM. Thansk @nspope !

…es; and sorting after simplify in preprocess_ts
@hyanwong hyanwong force-pushed the split-disjoint-fix branch from b888632 to 863bc35 Compare March 20, 2026 09:05
@nspope
Copy link
Contributor Author

nspope commented Mar 20, 2026

Commit you pushed will change the python version in the CI, is that intended @hyanwong ? Might be better to do that in a separate PR?

@hyanwong
Copy link
Member

Hmm, I just rebased from current main, or so I thought?

@nspope nspope added this pull request to the merge queue Mar 20, 2026
@nspope nspope removed this pull request from the merge queue due to a manual request Mar 20, 2026
@nspope nspope added this pull request to the merge queue Mar 20, 2026
@nspope
Copy link
Contributor Author

nspope commented Mar 20, 2026

The merge queue is stuck---at two attempts, tests all past but the lint check hangs on "waiting for status to be reported". I'm just going to bypass.

@nspope nspope removed this pull request from the merge queue due to a manual request Mar 20, 2026
@nspope nspope merged commit ee6082b into tskit-dev:main Mar 20, 2026
9 checks 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.

2 participants