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

0.6.2 release #417

Closed
wants to merge 8 commits into from
Closed

0.6.2 release #417

wants to merge 8 commits into from

Conversation

sanmitra
Copy link
Contributor

@sanmitra sanmitra commented Oct 11, 2019

Bug fixes

  • Fix ModelInsightsTest flakiness due to coefficients turning negative #407

New features/updates

  • Turn off logging of tokens #420
  • Regression training limit #413

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #417 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            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%) ⬆️

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.

@tovbinm
Copy link
Collaborator

tovbinm commented Oct 11, 2019

@sanmitra please update the version in all the other place (see #403)

@leahmcguire
Copy link
Collaborator

We need to remove the logging since I reverted Tuans change

@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @tovbinm to sign the Salesforce.com Contributor License Agreement.

@sanmitra sanmitra closed this Oct 14, 2019
@sanmitra sanmitra reopened this Oct 14, 2019
@sanmitra
Copy link
Contributor Author

@tovbinm @leahmcguire please review again. Thanks

@tovbinm
Copy link
Collaborator

tovbinm commented Oct 17, 2019

@sanmitra Have you published the artifacts already? I do not see them neither on bintray nor on central.

@tovbinm
Copy link
Collaborator

tovbinm commented Nov 14, 2019

@sanmitra Ping ;)

@sanmitra
Copy link
Contributor Author

@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.

@sanmitra sanmitra deleted the 0.6.2-release branch November 15, 2019 06:51
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.

None yet

3 participants