rrrlw/TDAstats

Support for other field coefficients

peekxc opened this issue · 6 comments

This is more of a question.

Is it in the scope of this package to re-adjust the ripser source code to support other coefficients in a prime field?

As you know, this should simply involve replacing areas like e.g. here with the code compiled when the USE_COEFFICIENTS preprocessor variable is enabled in the ripser package.

I imagine usage like

TDAstats::calculate_homology(unif2d, dim=1, p=11)

or something

rrrlw commented

Yes, particularly given that it's a Ripser feature, this would definitely be within the scope of the package. Unfortunately, I won't be able to get to adding it for a while - is this something you would be willing to add to TDAstats?

Seeing how I'll need it soon, yes I'll fork and PR soon-ish

rrrlw commented

Sounds good, thank you!

See PR #15 for code changes

rrrlw commented

Sorry I have been slow to respond, it looks like PR #15 has been closed for further editing. Let me know when it's ready or if there's anything with which I can help!

rrrlw commented

solved by #19, thank you @peekxc

code from #19 ported to rrrlw/ripserr (will add this as a TDAstats dependency after CRAN acceptance)