[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-20 Thread gaborhermann
Github user gaborhermann commented on the issue: https://github.com/apache/flink/pull/1849 Great. I agree this PR should be merged before #2838. @skonto thanks for taking up the review :) This is indeed a bit blocking. Hopefully I can improve upon #2838 next week, so by the

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-20 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/1849 Hi guys, my intention was to review #2838 but my feeling is that it overlaps with this one. @thvasilo we can push this one first as you said so I will have a look at it and comment on it. The benefit

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-20 Thread thvasilo
Github user thvasilo commented on the issue: https://github.com/apache/flink/pull/1849 Hello @gaborhermann. Personally I prefer to have PRs be as specific as possible, so I would recommend we try to get this merged before #2838, and then rebase that on master. Given the

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-20 Thread gaborhermann
Github user gaborhermann commented on the issue: https://github.com/apache/flink/pull/1849 Hi @skonto, I did not have time lately to finish up #2838, but I could clean it up next week. Although I believe this PR could be merged separately from mine. (Evaluating ranking

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-20 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/1849 @thvasilo thnx I will have a look --- 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] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-17 Thread thvasilo
Github user thvasilo commented on the issue: https://github.com/apache/flink/pull/1849 Hello @skonto this PR will probably be subsumed by #2838, you can check out the latest development there. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2017-01-17 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/1849 Hey @thvasilo is this under development? From what I see many other tasks depend on it right? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2016-10-04 Thread thvasilo
Github user thvasilo commented on the issue: https://github.com/apache/flink/pull/1849 @gaborhermann In terms of missing features, documentation is definitely missing, as @rawkintrevo mentioned. For the issues mentioned in the JIRA issue you linked I've replied on the dev

[GitHub] flink issue #1849: [FLINK-2157] [ml] Create evaluation framework for ML libr...

2016-10-04 Thread gaborhermann
Github user gaborhermann commented on the issue: https://github.com/apache/flink/pull/1849 Hi all, What is the status of this PR? It would be relevant for us, because we might like to use the evaluation framework proposed here. See