add site and mutation table rows

[bryo-han]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[gregorgorjanc]

@LynxJinyangii can you review this PR while I am away?

[hannesbecher]

I think @LynxJinyangii will be delighted to do so, @gregorgorjanc , as he is keen to base his additions on this PR.

[LynxJinyangii]

I've tested the functions with gh and eveything looks good to me.

[hannesbecher]

@gregorgorjanc During the pre-commit check, jarl now complains about the use of the triple colon. I think this may be a new behaviour of the recently released version 0.5.0 of jarl. This is to make sure that things don't break in the future in case internal naming changes. I guess we need to review (again) what we want to export...

[gregorgorjanc]

Yeah, I got the same error in previous PR merge too https://github.com/HighlanderLab/RcppTskit/actions/runs/23729353545/job/69119640231

We can tell jarl to ignore some lines. I have used that in a couple of places in the package.

I am sitting in the fence atm on wether we should or shouldn't export rtsk_* R&C++ functions. Let's wait a bit and see what the future brings;)

[gregorgorjanc]

@bryo-han @LynxJinyangii I have now done a polish of these add_row functions, so that we have the same style across all of them - more testing on R side than C++ side and consistent mapping with Python and C arg orders and types etc. Will wait for the GitHub Actions to run, but pre-commit and R CMD checks seems good already.