Vignette suggestions

[bryo-han]

"ts" -> "ts2" is important

[Copilot]

Pull request overview

Updates the introductory vignette to correctly reference the reloaded/simplified tree sequence object (ts2) and improves a couple of explanatory comments, aligning with the PR intent (“ts” -> “ts2” is important).

Changes:

• Fixes a vignette call to use ts2$num_individuals() after ts2 <- ts_load(ts_file).
• Rewords comments around reticulate/table-collection usage for clarity.
• Tweaks phrasing describing the equivalence between the Rcpp implementation and the method on TreeSequence.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[gregorgorjanc]

Thanks @hannesbecher! I squashed your commits, and ended up closing/destroying the PR, but since I did everything locally, I was still able to merge your code into main - see your commit at 24ba558

What I did was:

gh pr checkout 105 -R HighlanderLab/RcppTskit
git rebase upstream/main

git reset --soft HEAD~3
git commit -m "Vignette improvements"
git push --force-with-lease bryo-han main
# I reckon the top command must have closed/destroyed the PR (maybe because I am not the owner? - this resest/commit/push works well for me on my end to squash commits)

git switch main
git merge bryo-han/main
git push upstream