search

compTAG / r-tda

Other
4 stars 6 forks source link

first small pull request #16

Closed thomashli closed 5 years ago

thomashli commented 5 years ago

Added dionysus2 library as dionysus folder, swapped topology/simplex for dionysus2 simplex, added basic tests for kde and ripsfiltration.

dlm commented 5 years ago

@thomashli this looks like it is an older version of the current stab at getting Dionysus 2 integrated. Should I close this off are were you planning on coming back to this one also?

thomashli commented 5 years ago

You can close it off.

--Thomas

On Fri, Feb 22, 2019 at 8:13 PM David Millman notifications@github.com wrote:

@thomashli https://github.com/thomashli this looks like it is an older version of the current stab at getting Dionysus 2 integrated. Should I close this off are were you planning on coming back to this one also?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/compTAG/r-tda/pull/16#issuecomment-466614735, or mute the thread https://github.com/notifications/unsubscribe-auth/AU-Nff03uqzkLDJHUfz6REvNKZfRMiIeks5vQMAAgaJpZM4VU6au .

dlm commented 5 years ago

Trying a different direction