Fix issues with docstring of gtda.mapper.method_to_transform
#396
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wreise
approved these changes
Apr 14, 2020
ulupo
added a commit
that referenced
this pull request
Jun 2, 2020
* Fix issue with docstring example and document reshaping of 1D outputs (#396) * Toc (#394) * Update P landscapes Signed-off-by: ammedmar <[email protected]> * Add distances, inner products and kernels glossary entry Signed-off-by: ammedmar <[email protected]> * Remake vectorization changes Signed-off-by: ammedmar <[email protected]> * Change [] for \lbrack \rbrack Signed-off-by: ammedmar <[email protected]> * Update after W's comments Signed-off-by: ammedmar <[email protected]> * Update after W's comments Signed-off-by: ammedmar <[email protected]> * Update afte W's second comments Signed-off-by: ammedmar <[email protected]> * Update after Umbe's comments Signed-off-by: ammedmar <[email protected]> * Update after Umbe's second comments Signed-off-by: ammedmar <[email protected]> * Update after Umbe's third comments Signed-off-by: ammedmar <[email protected]> * Update after Umbe's 4th comments Signed-off-by: ammedmar <[email protected]> * Remove concept k-skeleton Signed-off-by: ammedmar <[email protected]> * Add table of content Signed-off-by: ammedmar <[email protected]> * Add Lp & lp. Update landscape Signed-off-by: ammedmar <[email protected]> * Update TOC indentation Signed-off-by: ammedmar <[email protected]> * Add heat vectorizations entry Signed-off-by: ammedmar <[email protected]> * Update indentation of TOC Signed-off-by: ammedmar <[email protected]> * Fix extra spacing in bibliography Signed-off-by: ammedmar <[email protected]> * Update bibliography hack for caps Signed-off-by: ammedmar <[email protected]> * After Umbe's Comments Signed-off-by: ammedmar <[email protected]> * Update after issue #398 Signed-off-by: ammedmar <[email protected]> * Update after Umbe's comments Signed-off-by: ammedmar <[email protected]> * Update after Umbe's 2nd comments Signed-off-by: ammedmar <[email protected]> Co-authored-by: ammedmar <[email protected]> * Speedup windows pipeline (#402) * Improve boost location for azure pipeline on windows The boost version installed in the pipeline is now used Signed-off-by: julian <[email protected]> Co-authored-by: Umberto Lupo <[email protected]> * Make bindings public (#395) * Make bindings public Signed-off-by: Guillaume Tauzin <[email protected]> * Fix pipeline on Mac (#407) * Refresh ccache * Enforce CXX standard to 14 on each module * Change variable name to comply with E741 Signed-off-by: julian <[email protected]> Co-authored-by: Umberto Lupo <[email protected]> * Mapper visualisation refactor: fix bugs, add summary statistics in hovertext, improve opacity, remove matplotlib dependency, add node_scale kwarg, add clone_pipeline kwarg to interactive plots,restructure/rename plotly_kwargs, improve code (#406) * Refactor of `mapper/visualisation.py` and `mapper/utils/visualisation.py` - Removal of color scaling - More modularity - Variable and function name changes - Remove matplotlib functions used for hoverlabel background color - Remove cmin and cmax - Change return signature of some functions * Add test-output.xml to .gitignore * Replace "text" key in plot_options["node_trace"] with "hovertext" * Improve width and opacity of nodes and edges * Display summary statistics in hovertext with significant figure rounding via new n_sig_figs kwarg * Rename plotly_kwargs to plotly_params * Remove matplotlib installation requirement * Fix mapper notebook * Add small comment on use of matplotlib in voids_on_the_plane * Place matplotlib in examples requirement in setup.py * Improve docstring of make_mapper_pipeline * Improve examples for make_mapper_pipeline and create one for plot_static_mapper_graph * Hide set_node_sizeref, add node_scale kwarg, add clone_pipeline kwarg to plot_interactive_mapper_graph - Update docstrings - Update quickstart notebook * Add helper function for colorscale-based interpolations, improve use of plotly_params Static and interactive plots display the expect hoverlabel colors in 3D, or white if things go wrong. Thanks to @lewtun for pointing out that this was needed. * Improve docstrings * Hide visualization module in mapper/utils * Add pip install matplotlib to notebook tests in manylinux job (#410) * Fix mapper docstring issues following #406 (#411) * Fix issues with mapper docs following #406 * Create giotto-tda version 0.2.2 (#413) * Turn CODE_OWNERS and CODE_AUTHORS into an rst file * Bump version number to 0.2.2 * Add release notes for v0.2.2 Co-authored-by: Anibal M. Medina-Mardones <[email protected]> Co-authored-by: ammedmar <[email protected]> Co-authored-by: REDS institute <[email protected]> Co-authored-by: Guillaume Tauzin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Types of changes
Description
The current docstring example for the
method_to_transformdecorator ingtda.mapperbecame incorrect following changes that reshape the output of the newtransformmethod to ensure it is at least 2D. This PR documents this reshaping in the docstring and fixes the example.Checklist
flake8to check my Python changes.I have added tests to cover my changes.pytestto check this on Python tests.