[GitHub] spark pull request #20897: [MINOR][DOC] Fix a few markdown typos
Github user Lemonjing commented on a diff in the pull request: https://github.com/apache/spark/pull/20897#discussion_r178481268 --- Diff: docs/mllib-pmml-model-export.md --- @@ -7,15 +7,15 @@ displayTitle: PMML model export - RDD-based API * Table of contents {:toc} -## `spark.mllib` supported models --- End diff -- backquotes in mds cause display problems (see http://spark.apache.org/docs/latest/mllib-pmml-model-export.html) --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20897: [MINOR][DOC] Fix a few markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20897 see http://spark.apache.org/docs/latest/ml-features.html#elementwiseproduct --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #20897: [MINOR][DOC] Fix a few markdown typos
Github user Lemonjing commented on a diff in the pull request: https://github.com/apache/spark/pull/20897#discussion_r177113075 --- Diff: docs/mllib-feature-extraction.md --- @@ -278,8 +278,8 @@ for details on the API. multiplication. In other words, it scales each column of the dataset by a scalar multiplier. This represents the [Hadamard product](https://en.wikipedia.org/wiki/Hadamard_product_%28matrices%29) between the input vector, `v` and transforming vector, `scalingVec`, to yield a result vector. -Qu8T948*1# --- End diff -- @HyukjinKwon If the original author did not reply, you can look at my latest reply and confirm this confused character sequence. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20897: [MINOR][DOC] Fix a few markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20897 @HyukjinKwon The description of ElementwiseProduct is obvious, and i think "`Qu8T948*1#`" is a mistake, if not, "`Qu8T948*1#`" is a input vector and 1 is scaling vector, and this is not a good example to express vector, even so, there is markdown syntax error. By the way, there is no example context here, such as "e.g."ï¼"for example" or something else. And i have queried the history and found another evidence. Before spark v1.6.0, this file do not have "`Qu8T948*1#`". So this maybe a mistake in one commit at v1.6.0 and this mistake not be found or fixed across v1.6.0 to now. ... https://github.com/apache/spark/blob/v1.4.1/docs/mllib-feature-extraction.md no https://github.com/apache/spark/blob/v1.5.0/docs/mllib-feature-extraction.md no https://github.com/apache/spark/blob/v1.5.2/docs/mllib-feature-extraction.md no https://github.com/apache/spark/blob/v1.6.0/docs/mllib-feature-extraction.md yes 1.6.0 ~ 2.3.0 yes Thanks, and hope to confirm it. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing closed the pull request at: https://github.com/apache/spark/pull/20889 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20889 @felixcheung thanks a lot. and i closed it. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20889 can someone help me to close this pr? i have merged these commits to a new pr. https://github.com/apache/spark/pull/20897 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #20897: [MINOR][DOC] Fix a few markdown typos
GitHub user Lemonjing opened a pull request: https://github.com/apache/spark/pull/20897 [MINOR][DOC] Fix a few markdown typos ## What changes were proposed in this pull request? Easy fix in the markdown. ## How was this patch tested? jekyII build test manually. Please review http://spark.apache.org/contributing.html before opening a pull request. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Lemonjing/spark master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/20897.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #20897 commit 937bbef522eedddbcb502f7f9692564040a63cd7 Author: lemonjing <932191671@...> Date: 2018-03-24T04:45:43Z [MINOR][DOC] Fix a few markdown typos --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20889 @HyukjinKwon Thanks a lot, could u help me to review these commits if u have timeã --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20889 @felixcheung hiï¼In my reading processï¼I found another md file typos and fix it. Do I need to close this pr and merge it into one commit and give a new pr? Thanks. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20889: [MINOR][DOC] Fix ml-guide markdown typos
Github user Lemonjing commented on the issue: https://github.com/apache/spark/pull/20889 @felixcheung Yes, I read the entire docs of Spark Mlib. There is no problem with other mds (I may not have seen it or other contributors solved it). This error is obvious, and it affects the link of the later issue, so i found it. [http://spark.apache.org/docs/latest/ml-guide.html#breaking-changes](url) --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request #20889: [MINOR][DOC] Fix ml-guide markdown typos
GitHub user Lemonjing opened a pull request: https://github.com/apache/spark/pull/20889 [MINOR][DOC] Fix ml-guide markdown typos ## What changes were proposed in this pull request? Easy fix in the markdown. ## How was this patch tested? Tested manually Please review http://spark.apache.org/contributing.html before opening a pull request. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Lemonjing/spark master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/20889.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #20889 commit be9cadb83fadae8099afe050ef8f99c7589e942a Author: lemonjing <932191671@...> Date: 2018-03-23T05:48:42Z [MINOR][DOC] Fix ml-guide markdown typos --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org