Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
Merging with master
Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #3547 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3547/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #3547 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3547/testReport)**
for PR 15018 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
Sounds good. I'll run fresh tests before merging to be safe though.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15018
@jkbradley I'll merge if there are no other thoughts here. Seems OK to me
too
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
Thanks for the updates! This LGTM, except for deciding about negative
weights.
Responding to your comment above, negative weights are just as problematic
as 0 weights. See my comment
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Looks like the build completed successfully. Somehow Jenkins didn't post
the message.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71329 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71329/testReport)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
My thought was that there's nothing in the computation that prevents it
(unlike 0 weights, which cause NaNs). Why not err on the side of maximum
flexibility to the end user? Just because there's no
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
@zapletal-martin Pinging since you wrote the original PR: There's
discussion here about whether IsotonicRegression should support negative
weights. Is there a good reason to? I haven't seen
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
@neggert I'll ask @mengxr about the negative weights since he oversaw the
original work here.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71271/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71271 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71271/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71271 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71271/testReport)**
for PR 15018 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15018
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71092/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71092 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71092/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71092/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71087/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71087/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #71087 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71087/testReport)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Your changes sound fine. I don't have strong feelings one way or another,
although I think we should at least throw a warning if we're discarding points.
I do want to point out that disallowing
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15018
SGTM. See if @jkbradley has more comment?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70886/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70886 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70886/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70886 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70886/testReport)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Done. It was surprisingly tricky to test, since the exception is thrown in
the executor.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70836/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70836 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70836/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70836 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70836/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70833 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70833/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70833/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70833 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70833/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70831/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70831 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70831/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70831 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70831/testReport)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Sorry, I've been on vacation. I'll make the requested changes in the next
day or two.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15018
Ping @neggert
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15018
@neggert I think this is looking good except for the question about zero
weights. If throwing an error is OK to you go for it. The test should be
restored, but be updated to expect an exception.
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15018
@neggert I am fine for throwing an error.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
@viirya Better to remove them, or throw an error? Personally, I'd rather be
alerted that I'm passing invalid input, rather than have it "fixed" for me.
---
If your project is set up for it, you
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/15018
For the zero-weight values, can we do similar to scikit-learn to remove
zero-weight values, like
https://github.com/amueller/scikit-learn/commit/2415100f79293bbbf52c12c36d63a6cf602cf3c4
---
If
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
@srowen Addressed your comments and fixed some style issues.
Some updated timings:
Alternating decreasing input
val x = (1 to length).toArray.map(_.toDouble)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70364/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70364/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70364/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70363/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70363/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70363/testReport)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70216 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70216/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70216/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70216 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70216/testReport)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Alright, the PAV algorithm has been completely re-written to follow what's
outlined in "Minimizing Separable Convex Functions Subject to Simple Chain
Constraints". I've tested it with a bunch of
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70214 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70214/testReport)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70214/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70214 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70214/testReport)**
for PR 15018 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15018
Just add commits here.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Found another input that triggers non-polynomial time with the code in this
PR. I'm again borrowing from scikit-learn. I think this is the case they found
that led them to re-write their
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15018
If it's correct and faster, I tend to favor merging this. The code looks OK
to my understanding and passes existing tests.
---
If your project is set up for it, you can reply to this email and have
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Looks like the new scikit-learn implementation suffers from a similar
problem to the one that the original Spark implementation had. I left them a
note pointing it out.
Right now I'm
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70136/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70136 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70136/consoleFull)**
for PR 15018 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70134/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70134/consoleFull)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70136 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70136/consoleFull)**
for PR 15018 at commit
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Given that the scikit-learn implementation I based this on has changed
(https://github.com/scikit-learn/scikit-learn/pull/7444) since I submitted the
PR, and @jkbradley has pointed out a reference
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #70134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70134/consoleFull)**
for PR 15018 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15018
Thanks for pinging about this---it looks important!
One request: Do you know if there is a resource with a clearer statement of
this precise algorithm for the L2 norm? That'd be nice to
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Anyone? @srowen?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user neggert commented on the issue:
https://github.com/apache/spark/pull/15018
Could someone please take a look at this? @mengxr maybe? Looks like you
merged the initial implementation.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/65117/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15018
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #65117 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65117/consoleFull)**
for PR 15018 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15018
**[Test build #65117 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/65117/consoleFull)**
for PR 15018 at commit
87 matches
Mail list logo