8000 [MRG] Modify Svm examples for matplotlibv2 comp by rishikksh20 · Pull Request #8456 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]

Skip to content

[MRG] Modify Svm examples for matplotlibv2 comp #8456

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

Merged
merged 8 commits into from
Mar 4, 2017
Merged

[MRG] Modify Svm examples for matplotlibv2 comp #8456

merged 8 commits into from
Mar 4, 2017

Conversation

rishikksh20
Copy link
Contributor
@rishikksh20 rishikksh20 commented Feb 25, 2017

Reference Issue

#8364

What does this implement/fix? Explain your changes.

Add edgecolors attribute in scatter plot.

Any other comments?

Please verify all example plots using following links :

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: #8364
Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: #8364
Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: #8364
Add `edgecolors` attribute to scatter plot for better visualization.

Issue: #8364
Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: #8364
Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: #8364
Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: #8364
Remove extra white space.

Issue: #8364
@codecov
Copy link
codecov bot commented Feb 25, 2017

Codecov Report

Merging #8456 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8456   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files         342      342           
  Lines       60907    60907           
=======================================
  Hits        58154    58154           
  Misses       2753     2753

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 067adad...842b90b. Read the comment docs.

1 similar comment
@codecov
Copy link
codecov bot commented Feb 25, 2017

Codecov Report

Merging #8456 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8456   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files         342      342           
  Lines       60907    60907           
=======================================
  Hits        58154    58154           
  Misses       2753     2753

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 067adad...842b90b. Read the comment docs.

@jnothman
Copy link
Member

In plot_separating_hyperplane_unbalanced.py the markers appear larger, and the figure smaller, making it all somewhat confused.

@jnothman
Copy link
Member

Otherwise LGTM

@tguillemot
Copy link
Contributor

The figure in plot_oneclass.py is also smaller. Can you change that too ?

@rishikksh20
Copy link
Contributor Author

Actually the problem is in embedding of plot in html. Whenever I run the script in python console it runs fine in both scenario (matplotlib v2 and matplotlib older version (v 0.15)) but when I ran the same in Ipython notebook the visualisation size of the plot is not same ( matplotlib v2 plot looks little smaller in size than older version ) .

@amueller
Copy link
Member
amueller commented Mar 4, 2017

Could that also be an issue of updated sphinx-gallery?

Btw I feel like the last one didn't re 10000 ally need "fixing" but I'm fine with it.

@amueller
Copy link
Member
amueller commented Mar 4, 2017

LGTM, I like incremental improvements.

@amueller amueller merged commit cdd693b into scikit-learn:master Mar 4, 2017
@rishikksh20 rishikksh20 deleted the svm_plot_matplotlibv2_comp branch March 10, 2017 07:46
@Przemo10 Przemo10 mentioned this pull request Mar 17, 2017
herilalaina pushed a commit to herilalaina/scikit-learn that referenced this pull request Mar 26, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
massich pushed a commit to massich/scikit-learn that referenced this pull request Apr 26, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
* Modify plot_custom_kernel for matplotlib v2 comp

Add `edgecolors` attribute in scatter plot for better visualization
in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_oneclass.py for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization in matplotlib version 2

Issue: scikit-learn#8364

* Modify plot_rbf_parameters for matplotlib v2

Add `edgecolors` attribute to scatter plot for
better visualization.

Issue: scikit-learn#8364

* Modify plot_separating_hyperplane_unbalanced for matplotlib v2

Add `edgecolors` attribute to scatter plot for better visualization.

Issue: scikit-learn#8364

* Modify plo_svm_kernels for matplotlib v2

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_margin for matplotlib v2 comp

Add `edgecolors` attribute to scatter plot for better
visualization.

Issue: scikit-learn#8364

* Modify plot_svm_nonlinear for matplotlib v2

Add `edgecolors` attribute to scatter plot for matplotlib
version 2 compatibility

Issue: scikit-learn#8364

* Modify file for remove flake8 error

Remove extra white space.

Issue: scikit-learn#8364
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.

4 participants
0