Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create v0.2.1 #392

Merged
merged 9 commits into from Apr 8, 2020
Merged

Create v0.2.1 #392

merged 9 commits into from Apr 8, 2020

Conversation

ulupo
Copy link
Collaborator

@ulupo ulupo commented Apr 8, 2020

Merge from master to create release v0.2.1.

ulupo and others added 9 commits March 23, 2020 15:43
* Add @rth to code authors and list of contributors for v0.2.0
* Add twine check to azure-pipelines.yml

* Fix formatting and duplicate issues in README.rst
…erage (#384)

* Remove images subpackage from code omitted from coverage

* Add tests for Filtering class

* Add tests in diagrams subpackage, linting

- Rename test_features -> test_features_representations
- Create parametrized dummy tests for fit_transform_plot method of transformers in representations.py
- Improve structure and range of tests in test_preprocessing in a similar way
- Create dummy tests of shape for BettiCurve and PersistenceLandscape
- Linting and code style changes elsewhere

* Add fit_transform_plot test for GraphGeodesicDistance

* Add fit_transform_plot tests in gtda.homology

* Fix typo in voids_on_the_plane notebook

* Add fit_transform_plot tests for images subpackage

* Add fit_transform_plot tests for point_clouds subpackage

* Add test for SlidingWindow.plot

* Linting

* Exclude gtda.utils._doc and gtda.utils.intervals from coveragerc

* Implement @wreise's suggestions

- Rename some test functions
- Implement a solution to the automatic tab opening problem, based on setting the default value of plotly.io.renderers in test files

* Upload coverage report only from Azure macOS job

* Exclude test folders from coverage reports
* Update P landscapes

* Add distances, inner products and kernels glossary entry

Signed-off-by: ammedmar <anibal@medina-mardones.com>
* Update ccache

* Update variable used to indicate boost headers location

Official answer from boost devs: https://discourse.cmake.org/t/findboost-difference-between-boost-include-dir-and-boost-include-dirs/855

* Add install boost step in Azure CI, which downloads boost

Signed-off-by: julian <julian.burellaperez@heig-vd.ch>
…/refactor check_point_clouds and add tests (#386)

* Remove boolean mask on infinite values in VietorisRipsPersistence

* Refactoring of check_point_clouds

- Rename distance_matrix kwarg to distance_matrices
- Add checks for square input when distance_matrices is set to True
- More comprehensive/friendly error messages and warnings

* Increase generality of ripser function when metric == 'precomputed'

Sklearn's pairwise_distances is no longer called when metric='precomputed'.

This is because pairwise_distances checks for positivity and finite input in that case, but this is unnecessarily restrictive for ripser. 

pairwise_distances also checks for square input so this is no longer checked in the case metric='precomputed'.  But the ripser function is not meant for use in isolation. It is meant to be called by a VietorisRipsPersistence instance, and input validation can be offloaded to that class. In particular, 9bab872 introduces checks for square input.

* Add tests for check_point_clouds

* Refresh docstrings for VietorisRipsPersistence

- Mention "adjacency matrices of weighted undirected graphs" as well as distance matrices
- Mention compatibility with filtration in docstrings for fit and transform
* Add citation section in README

* Add citation info to FAQ page in docs

Signed-off-by: Guillaume Tauzin <guillaumetauzin.ut@gmail.com>

Co-authored-by: Umberto Lupo <46537483+ulupo@users.noreply.github.com>
* Fix citing giotto-tda section
* Fix Important link section
@ulupo ulupo merged commit 08619b0 into 0.2.X Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants