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

DOC improve examples styles #813

Merged

Conversation

glemaitre
Copy link
Member

Improve the example style with newer marker.

@pep8speaks
Copy link

pep8speaks commented Feb 16, 2021

Hello @glemaitre! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 23:1: E402 module level import not at top of file
Line 63:1: E402 module level import not at top of file

Line 17:1: E402 module level import not at top of file
Line 28:1: E402 module level import not at top of file
Line 29:1: E402 module level import not at top of file
Line 37:1: E402 module level import not at top of file
Line 61:1: E402 module level import not at top of file
Line 69:1: E402 module level import not at top of file
Line 85:1: E402 module level import not at top of file
Line 122:1: E402 module level import not at top of file

Line 29:1: E402 module level import not at top of file
Line 46:1: E402 module level import not at top of file
Line 64:1: E402 module level import not at top of file
Line 65:1: E402 module level import not at top of file
Line 100:1: E402 module level import not at top of file
Line 123:1: E402 module level import not at top of file
Line 124:1: E402 module level import not at top of file
Line 125:1: E402 module level import not at top of file
Line 126:1: E402 module level import not at top of file
Line 142:1: E402 module level import not at top of file
Line 143:1: E402 module level import not at top of file
Line 157:1: E402 module level import not at top of file
Line 180:1: E402 module level import not at top of file
Line 181:1: E402 module level import not at top of file
Line 261:1: E402 module level import not at top of file
Line 262:1: E402 module level import not at top of file
Line 317:1: E402 module level import not at top of file
Line 340:1: E402 module level import not at top of file
Line 341:1: E402 module level import not at top of file

Line 32:1: E402 module level import not at top of file
Line 33:1: E402 module level import not at top of file
Line 51:1: E402 module level import not at top of file
Line 58:1: E402 module level import not at top of file
Line 59:1: E402 module level import not at top of file
Line 68:1: E402 module level import not at top of file
Line 69:1: E402 module level import not at top of file
Line 86:1: E402 module level import not at top of file
Line 87:1: E402 module level import not at top of file
Line 115:1: E402 module level import not at top of file
Line 135:1: E402 module level import not at top of file
Line 136:1: E402 module level import not at top of file
Line 183:1: E402 module level import not at top of file
Line 184:1: E402 module level import not at top of file

Line 23:11: F821 undefined name 'version'

Comment last updated at 2021-02-16 22:48:20 UTC

@codecov
Copy link

codecov bot commented Feb 16, 2021

Codecov Report

Merging #813 (0debe32) into master (9f4173d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #813   +/-   ##
=======================================
  Coverage   98.59%   98.59%           
=======================================
  Files          93       93           
  Lines        6036     6036           
  Branches      503      503           
=======================================
  Hits         5951     5951           
  Misses         84       84           
  Partials        1        1           

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 9f4173d...0debe32. Read the comment docs.

@glemaitre glemaitre merged commit 2150e67 into scikit-learn-contrib:master Feb 16, 2021
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

2 participants