-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
[MRG] improve tfidfvectorizer documentation #12822
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
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
7ee45a3
cleaning up flake8 errors in text.py
8153489
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn
2a34a37
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
453f1f4
fixed flake8 errors in tdidf description
25da828
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
d9cadb8
put tf-idf formula in a math block'
df770cc
moved CountVec and TfidfT desc from class to 'parameter' section
27907c1
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
1492f5c
remove blank spaces
3a2c30f
adding space after colon
3c74ea7
formatting the :math: expressoin
c020cbb
still fixing formatting
70cf279
still fixing formatting
a70893c
still fixing formatting
5bf1808
remove :math: and do step-by-step check of docstring error
72fe738
added 1 :math: block in; this should work
d4c14b3
added 1 :math: block in; this should work
14c5186
delete line, move (if ``smooth_id` up)
4feee7b
for CountVectorizer, added parameter type
c451532
for TfidfTransformer, added parameter type
4c6eaf0
moved CountV and TfidfT from 'Parameters' to 'See also'
403acf3
expanded the definition of 'norm' in Parameters section
a289dc8
added more defaults to parameters; added math block
1997b9d
added 3rd math block
729d1fd
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
6a1c358
put all defaults in () & added \log to :math:
ea9066b
changed 'log' to '\log' in math formula
cef1b84
changing '\log' to 'log' because of CI errors
51990e5
trying to fix math that is not rendering
482be9b
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
a3acfe0
removed '\text' from math formula
98defdd
iterative work: remove 2 & 3 math blocks
d4b3493
changed { to ( in math expression
5feaf73
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
674135c
removed math blocks; norm def in class TfidfV
42c4532
shortened definitions
a403b1e
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
c957f69
minor text edits
9d5f842
cleaning up, formatting
c62539a
Merge branch 'master' of https://github.com/scikit-learn/scikit-learn…
83c534a
single backticks around the class names for hyperlinking'
32c1eea
set to ':class:'
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs a blank line before it. See broken rendering at https://42814-843222-gh.circle-artifacts.com/0/doc/modules/generated/sklearn.feature_extraction.text.TfidfVectorizer.html