Add pip install matplotlib to notebook tests in manylinux job #410
Conversation
This file contains 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
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 <anibal@medina-mardones.com> * Add distances, inner products and kernels glossary entry Signed-off-by: ammedmar <anibal@medina-mardones.com> * Remake vectorization changes Signed-off-by: ammedmar <ammedmar@gmail.com> * Change [] for \lbrack \rbrack Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after W's comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after W's comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update afte W's second comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's second comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's third comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's 4th comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Remove concept k-skeleton Signed-off-by: ammedmar <ammedmar@gmail.com> * Add table of content Signed-off-by: ammedmar <ammedmar@gmail.com> * Add Lp & lp. Update landscape Signed-off-by: ammedmar <ammedmar@gmail.com> * Update TOC indentation Signed-off-by: ammedmar <ammedmar@gmail.com> * Add heat vectorizations entry Signed-off-by: ammedmar <ammedmar@gmail.com> * Update indentation of TOC Signed-off-by: ammedmar <ammedmar@gmail.com> * Fix extra spacing in bibliography Signed-off-by: ammedmar <ammedmar@gmail.com> * Update bibliography hack for caps Signed-off-by: ammedmar <ammedmar@gmail.com> * After Umbe's Comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after issue #398 Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's comments Signed-off-by: ammedmar <ammedmar@gmail.com> * Update after Umbe's 2nd comments Signed-off-by: ammedmar <ammedmar@gmail.com> Co-authored-by: ammedmar <anibal@medina-mardones.com> * 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 <julian.burellaperez@heig-vd.ch> Co-authored-by: Umberto Lupo <46537483+ulupo@users.noreply.github.com> * Make bindings public (#395) * Make bindings public Signed-off-by: Guillaume Tauzin <guillaumetauzin.ut@gmail.com> * 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 <julian.burellaperez@heig-vd.ch> Co-authored-by: Umberto Lupo <46537483+ulupo@users.noreply.github.com> * 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 <ammedmar@gmail.com> Co-authored-by: ammedmar <anibal@medina-mardones.com> Co-authored-by: REDS institute <reds-heig@users.noreply.github.com> Co-authored-by: Guillaume Tauzin <guillaumetauzin.ut@gmail.com>
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.
#406 broke the manylinux notebook tests as it removed
matplotlibfrom the library requirements. This PR adds a pip install matplotlib step in the CI to fix this.