Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to Spark 2.4.3 + XGBoost 0.90 + MLeap 0.14 #327

Merged
merged 31 commits into from
Jul 16, 2019
Merged

Conversation

tovbinm
Copy link
Collaborator

@tovbinm tovbinm commented May 30, 2019

Related issues
We would like to use XGBoost 0.90, but it requires Spark 2.4.x and also #184

Describe the proposed solution

Describe alternatives you've considered
NA

@codecov
Copy link

codecov bot commented May 30, 2019

Codecov Report

Merging #327 into master will increase coverage by 0.1%.
The diff coverage is 88.16%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #327     +/-   ##
=========================================
+ Coverage   86.68%   86.79%   +0.1%     
=========================================
  Files         336      336             
  Lines       10821    10895     +74     
  Branches      359      562    +203     
=========================================
+ Hits         9380     9456     +76     
+ Misses       1441     1439      -2
Impacted Files Coverage Δ
...rce/op/stages/impl/feature/ScalerTransformer.scala 96.29% <ø> (ø) ⬆️
...m/salesforce/op/aggregators/ExtendedMultiset.scala 75% <0%> (ø) ⬆️
...n/scala/com/salesforce/op/dsl/RichMapFeature.scala 67.64% <0%> (ø) ⬆️
...a/com/salesforce/op/filters/PreparedFeatures.scala 80.48% <100%> (ø) ⬆️
...stages/impl/feature/TimePeriodMapTransformer.scala 100% <100%> (ø) ⬆️
...s/impl/feature/EmailToPickListMapTransformer.scala 100% <100%> (ø) ⬆️
...op/evaluators/OpMultiClassificationEvaluator.scala 94.73% <100%> (ø) ⬆️
.../scala/com/salesforce/op/dsl/RichDateFeature.scala 95.45% <100%> (ø) ⬆️
...rce/op/stages/impl/feature/PhoneNumberParser.scala 100% <100%> (ø) ⬆️
...om/salesforce/op/utils/spark/OpSparkListener.scala 98.6% <100%> (+1.19%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fd440b7...0aa80f9. Read the comment docs.

Jauntbox and others added 5 commits May 30, 2019 23:23
… made to decision tree pruning in Spark 2.4. If nodes are split, but both child nodes lead to the same prediction then the split is pruned away. This updates the test so this doesn't happen for feature 'b'
@wsuchy
Copy link
Contributor

wsuchy commented May 31, 2019

LGTM

Copy link
Contributor

@gerashegalov gerashegalov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tovbinm tovbinm changed the title Update to Spark 2.4.3 and XGBoost 0.90 Update to Spark 2.4.3 + XGBoost 0.90 + MLeap 0.14 Jun 24, 2019
@leahmcguire
Copy link
Collaborator

LGTM

@tovbinm tovbinm merged commit 3e02bf7 into master Jul 16, 2019
@tovbinm tovbinm deleted the mt/spark-2.4 branch July 16, 2019 02:34
@tovbinm tovbinm restored the mt/spark-2.4 branch August 30, 2019 20:38
tovbinm added a commit that referenced this pull request Aug 30, 2019
@tovbinm tovbinm mentioned this pull request Aug 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants