giotto-ai / giotto-tda

A high-performance topological machine learning toolbox in Python
https://giotto-ai.github.io/gtda-docs
Other
845 stars 173 forks source link

Fix read-only error analogous to #427 in Amplitude and PairwiseDistance #481

Closed ulupo closed 4 years ago

ulupo commented 4 years ago

Reference issues/PRs

428 and #427

Types of changes

Description Extends #428, fixing the analogous issue to #427 in Amplitude and PairwiseDistance, and adds a regression test. Also improves the handling of edge cases (zero range of births or persistences) in the persistence_images function in gtda/diagrams/_metrics.py.

Checklist