Closed sanmitra closed 5 years ago
Merging #417 into master will increase coverage by
0.01%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #417 +/- ##
==========================================
+ Coverage 86.93% 86.95% +0.01%
==========================================
Files 337 337
Lines 11100 11100
Branches 366 366
==========================================
+ Hits 9650 9652 +2
+ Misses 1450 1448 -2
Impacted Files | Coverage Δ | |
---|---|---|
...es/src/main/scala/com/salesforce/op/OpParams.scala | 89.79% <0%> (+4.08%) |
:arrow_up: |
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 ccc1501...15dcf10. Read the comment docs.
@sanmitra please update the version in all the other place (see #403)
We need to remove the logging since I reverted Tuans change
Thanks for the contribution! Before we can merge this, we need @tovbinm to sign the Salesforce.com Contributor License Agreement.
@tovbinm @leahmcguire please review again. Thanks
@sanmitra Have you published the artifacts already? I do not see them neither on bintray nor on central.
@sanmitra Ping ;)
@tovbinm we can close this PR. I made this PR just to release Turn off logging of tokens #420
for E1 AutoML as it could have caused security issue. But instead of public release, we just made an internal release for E1 AutoML thanks to Gera for setting that up. Anyways, we are planning on a separate public release soon.
Bug fixes
ModelInsightsTest
flakiness due to coefficients turning negative #407New features/updates