[docs] Change some 'parallel learning' references to 'distributed learning' #4000
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
jameslamb
requested review from
btrotta,
chivee,
henry0312,
huanzhang12,
Laurae2 and
wxchan
as code owners
jameslamb
added a commit
that referenced
this pull request
Feb 19, 2021
StrikerRUS
approved these changes
Feb 19, 2021
| @@ -21,14 +21,14 @@ LightGBM is a gradient boosting framework that uses tree based learning algorith | |||
| - Faster training speed and higher efficiency. | |||
| - Lower memory usage. | |||
| - Better accuracy. | |||
| - Support of parallel and GPU learning. | |||
| - Support of parallel, distributed, and GPU learning. | |||
There was a problem hiding this comment.
Same change here please:
https://github.com/microsoft/LightGBM/blame/master/docs/index.rst#L20
There was a problem hiding this comment.
ah missed that one, thanks. Added in 7985222
jameslamb
added a commit
that referenced
this pull request
Feb 22, 2021
…3951) * rework distributed learning page * more references * more changes * more changes * add anchors for olds links * revert changes from #4000 * fix links * more links * Apply suggestions from code review Co-authored-by: Nikita Titov <[email protected]> * Update docs/Parallel-Learning-Guide.rst Co-authored-by: Nikita Titov <[email protected]> Co-authored-by: Nikita Titov <[email protected]>
|
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
In #3596, we agreed to replace the terminology "parallel learning" with "distributed learning" in this project's documentation, everywhere that that phrase was used to refer to multi-machine training. It was also agreed in that issue that this change should not break any current documentation links.
I've started doing that in #3951, in preparation for adding new Dask docs (#3814).
While working on #3951, I realized that many of the places where this language needs to be changed won't break any links. These are in things like code comments and log messages.
This pull request proposes just those changes that don't break links, to hopefully reduce the size of #3951 and make it easier to review.