search

rstudio / mleap

R Interface to MLeap
http://spark.rstudio.com/guides/mleap/
Apache License 2.0
24 stars 9 forks source link

Update to mleap 0.16.0 #41

Closed tschutte closed 3 years ago

tschutte commented 4 years ago

This pull request updates the mleap version to 0.16.0.

tschutte commented 3 years ago

@kevinykuo, is it possible to have this merged? Or is there other work needed to make this happen?

yitao-li commented 3 years ago

@tschutte Hey sorry for the delay... The change looks good to me :+1: (as in, all tests passed after rebuilding the inst/java/mleap-*.jar files with the newer ml.combust.mleap:mleap-spark_2.11:0.16.0). Thanks for contributing to mleap!

I'll create another PR that keeps the build artifacts from inst/java/mleap-*.jar up-to-date.

Kevin is in the process of transferring ownership of https://github.com/rstudio/mleap to me BTW. So I'll be the new maintainer of mleap moving forward.

yitao-li commented 3 years ago

@tschutte FYI please re-run remotes::install_github("rstudio/mleap") to pick up updated jar files from https://github.com/rstudio/mleap/pull/43 if you want to start using mleap with ml.combust.mleap:mleap-spark_2.11:0.16.0 immediately.

tschutte commented 3 years ago

Excellent, thank you again.