JianGoForIt / YellowFin

auto-tuning momentum SGD optimizer
Apache License 2.0
422 stars 93 forks source link

Change license approval to integrate YF in T2T #16

Closed ReDeiPirati closed 7 years ago

ReDeiPirati commented 7 years ago

@JianGoForIt as i said in different issues i was trying to adapt YF to be usable in tensor2tensor and after my PR to definitively integrate YF in T2T, it raised a license problem. Once the PR is accepted it will override your MIT License, so the T2T authors need your OK(approval) to keep the PR, otherwise we cannot use your code. This is the PR.

JianGoForIt commented 7 years ago

Hi @ReDeiPirati

Please go ahead and override the license. I have checked it up with all collaborators and we are all happy with it.

Cheers