Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-219120765
bump? failing on flaky test, can someone restart/verify/etc?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rawkintrevo commented on a diff in the pull request:
https://github.com/apache/flink/pull/1898#discussion_r63605367
--- Diff: docs/apis/batch/libs/ml/cross_validation.md ---
@@ -0,0 +1,175 @@
+---
+mathjax: include
+title: Cross Validation
+
+# Sub
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-219910631
Thanks for the review @tillrohrmann
I need to find a markdown editor with spell check...
---
If your project is set up for it, you can reply to this email
Github user rawkintrevo commented on the issue:
https://github.com/apache/flink/pull/1898
No problem @tillrohrmann ! I know there's been a lot going on. thank you!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
GitHub user rawkintrevo opened a pull request:
https://github.com/apache/flink/pull/1384
Flink 1994- Add different gain calculation schemes to SGD
Added 5 gain calculation schemes and .setOptimization method which takes an
int values 0-5.
You can merge this pull request into
Github user rawkintrevo closed the pull request at:
https://github.com/apache/flink/pull/1384
---
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 feature
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1384#issuecomment-159139713
did what you said ( I think ) and opened a new pull request
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user rawkintrevo opened a pull request:
https://github.com/apache/flink/pull/1397
[FLINK-1994] [ml] Add different gain calculation schemes to SGD
Continuation of pull request 1384.
Fixed long line issue, rebased on top of current master, and changed title
per
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-159646696
That last fail looked like it happened in Kafka from something I never
touched? E.g. random. Is there anyway to just retry the travis-ci build?
---
If your project
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-159780302
So... looks like this passed. What do I do next?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-165941576
I think I really screwed this up... I was just trying to undo the changes
to flink-staging/flink-ml/pom.xml
---
If your project is set up for it, you can reply to
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-170162685
on your second point- I agree that Enum would be better for human readable
variable setting, but if one was doing a grid parameter search it would be
easier to
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-170257023
still refactoring to enumeration, but wanted to toss this up in case anyone
is watching- I added a decay parameter which significantly generalizes the Xu
and
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-173422504
I can't get the enumeration thing to work. If you can modify it that would
be awesome. Thanks.
---
If your project is set up for it, you can reply to this
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-173578713
For sure, definitely want to close the books on this one, and I appreciate
all your help and patience as I have been learning. I will work on this tonight.
---
If
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-173801417
Sorry for the multiple commits- It wasn't registering the changes here so I
kept pushing... I can squash them if you want.
I update the docs per
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1397#issuecomment-173893702
Thank you guys!
---
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
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/871#issuecomment-203067582
Continued from mailing list: Till already mentioned that having Rsquared
built in to MLR was just a convenience method, it's not good for a number of
reaso
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1412#issuecomment-203953994
@nikste awesome work, any updates?
---
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
GitHub user rawkintrevo opened a pull request:
https://github.com/apache/flink/pull/1865
[FLINK][3720] Added Warm Starts
This adds warm start functionality for IterativeSolver and
exposes/implements in Multiple Linear Regression.
Everything in the checklist should be GTG
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1865#issuecomment-207584968
There is an error in the docs (code isn't in the highlighting)- waiting to
see what else needs to be done (per reviewers comments) to minimize the co
GitHub user rawkintrevo opened a pull request:
https://github.com/apache/flink/pull/1875
[FLINK-3742] Add Multilayer Perceptron
https://en.wikipedia.org/wiki/Multilayer_perceptron
Multilayer perceptron is a simple sort of artificial neural network. It
creates a directed
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1875#issuecomment-209149108
Interesting. I still need to add some docs and squash my commits. I'll do
that in the next day or two and retrigger the CI build- maybe the issue will
GitHub user rawkintrevo opened a pull request:
https://github.com/apache/flink/pull/1898
[FLINK-2259][ml] Add Train-Testing Splitters
This PR adds an object in ml/pipeline called splitter with the following
methods:
randomSplit: Splits a DataSet into two data sets using
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1875#issuecomment-210727904
Failed on one build @ error:
scala.reflect.internal.MissingRequirementError: object scala.runtime in
compiler mirror not found.
---
If your project is set up for
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-210728264
One build failed on error: scala.reflect.internal.MissingRequirementError:
object scala.runtime in compiler mirror not found.
Another on some weird YARN
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-211507574
Any thoughts on this approach or go ahead and write docs/prepare to merge?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1909#issuecomment-211873070
Nice. If this gets merged before https://github.com/apache/flink/pull/1898
I'll integrate it in. Otherwise I'll open a seperate PR after.
---
If your
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1849#issuecomment-212563858
Are there going to be useage docs on this?
---
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 rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1849#issuecomment-212633912
Also two quick issues.
*pipelines*
```scala
val scaler = MinMaxScaler()
val pipeline = scaler.chainPredictor(mlr)
val evaluationDS
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1849#issuecomment-212916215
np, also RE: my comment on the docs- I think I can lend a hand there (I was
actually testing functionality to make sure I understood how it worked). Let me
know if
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1849#issuecomment-213411019
The transformer needs to scale the label too... I might not be correctly
understanding your last paragraph / what you are proposing.
I agree with paragraph
Github user rawkintrevo commented on a diff in the pull request:
https://github.com/apache/flink/pull/1898#discussion_r61503436
--- Diff:
flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/preprocessing/Splitter.scala
---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the
Github user rawkintrevo commented on a diff in the pull request:
https://github.com/apache/flink/pull/1898#discussion_r61504020
--- Diff:
flink-libraries/flink-ml/src/test/scala/org/apache/flink/ml/preprocessing/SplitterITSuite.scala
---
@@ -0,0 +1,73 @@
+/*
+ * Licensed
Github user rawkintrevo commented on a diff in the pull request:
https://github.com/apache/flink/pull/1898#discussion_r61582494
--- Diff:
flink-libraries/flink-ml/src/main/scala/org/apache/flink/ml/preprocessing/Splitter.scala
---
@@ -0,0 +1,215 @@
+/*
+ * Licensed to the
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-215826478
Hey @tillrohrmann, thanks for the review. I've addressed your commments in
the code. I am in the middle of documentation had to commit. Should finish up
docs
Github user rawkintrevo commented on the pull request:
https://github.com/apache/flink/pull/1898#issuecomment-216241912
This should be gtg
---
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
Github user rawkintrevo commented on the issue:
https://github.com/apache/flink/pull/1875
closing this
---
Github user rawkintrevo commented on the issue:
https://github.com/apache/flink/pull/1865
closing this
---
Github user rawkintrevo closed the pull request at:
https://github.com/apache/flink/pull/1875
---
Github user rawkintrevo closed the pull request at:
https://github.com/apache/flink/pull/1865
---
41 matches
Mail list logo