Merging a CL about language identification into M56 |
||||||
Issue descriptionWe'd like to merge the following CL into M56: https://codereview.chromium.org/2546943002/ The main change is a retrained model with additional script-based features.
,
Dec 3 2016
,
Dec 3 2016
[Automated comment] DEPS changes referenced in bugdroid comments, needs manual review.
,
Dec 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8377083c34ac86e133e527bacc2ae49a0ead53b5 commit 8377083c34ac86e133e527bacc2ae49a0ead53b5 Author: ynovikov <ynovikov@chromium.org> Date: Sun Dec 04 16:52:52 2016 Revert of Updating the CLD3 commit hash in DEPS (patchset #1 id:1 of https://codereview.chromium.org/2546943002/ ) Reason for revert: I believe this breaks https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Debug%20%28AMD%29/builds/398 base::WeakPtr<base::win::ObjectWatcher>::get [0x1035E083+67] invalid_parameter [0x66D62011+161] std::_String_const_iterator<std::_String_val<std::_Simple_types<char> > >::operator* [0x023398F3+211] std::_String_iterator<std::_String_val<std::_Simple_types<char> > >::operator* [0x0233994F+15] chrome_lang_id::FMLParser::NextItem [0x06832F16+726] chrome_lang_id::FMLParser::Initialize [0x06832B1F+111] chrome_lang_id::FMLParser::Parse [0x068330CF+31] chrome_lang_id::GenericFeatureExtractor::Parse [0x06835340+48] ?Setup@?$EmbeddingFeatureExtractor@V?$FeatureExtractor@VSentence@chrome_lang_id@@$$V@chrome_lang_id@@VSentence@2@$$V@chrome_lang_id@@UAEXPAVTaskContext@2@@Z [0x0681AD70+128] chrome_lang_id::NNetLanguageIdentifier::Setup [0x0681AE8C+28] chrome_lang_id::NNetLanguageIdentifier::NNetLanguageIdentifier [0x06815F2F+527] chrome_lang_id::NNetLanguageIdentifier::NNetLanguageIdentifier [0x06816009+25] translate::DeterminePageLanguage [0x06A3F017+1287] translate::DeterminePageLanguage [0x06A3EC50+320] translate::TranslateHelper::PageCaptured [0x06A3D13F+335] ChromeRenderFrameObserver::CapturePageText [0x06874729+489] ChromeRenderFrameObserver::DidMeaningfulLayout [0x06874CA9+73] content::RenderFrameImpl::DidMeaningfulLayout [0x12BA7F73+99] content::RenderWidget::didMeaningfulLayout [0x12C5FC09+169] blink::WebViewImpl::updateAllLifecyclePhases [0x1A84A852+690] blink::WebViewImpl::resizeViewWhileAnchored [0x1A844719+265] blink::WebViewImpl::resizeWithBrowserControls [0x1A844A8E+670] content::RenderViewImpl::ResizeWebWidget [0x12C39199+89] content::RenderWidget::Resize [0x12C5AB34+756] content::RenderWidget::OnResize [0x12C595F2+114] content::RenderViewImpl::OnResize [0x12C37F54+516] base::DispatchToMethodImpl<content::RenderWidget *,void (__thiscall content::RenderWidget::*)(content::ResizeParams const &),std::tuple<content::ResizeParams> const &,0> [0x12C4B006+54] base::DispatchToMethod<content::RenderWidget *,void (__thiscall content::RenderWidget::*)(content::ResizeParams const &),std::tuple<content::ResizeParams> const &> [0x12C4A505+53] IPC::DispatchToMethod<content::RenderWidget,void (__thiscall content::RenderWidget::*)(content::ResizeParams const &),void,std::tuple<content::ResizeParams> > [0x12C4AAD6+38] IPC::MessageT<ViewMsg_Resize_Meta,std::tuple<content::ResizeParams>,void>::Dispatch<content::RenderWidget,content::RenderWidget,void,void (__thiscall content::RenderWidget::*)(content::ResizeParams const &)> [0x12C47F5C+300] content::RenderWidget::OnMessageReceived [0x12C580F7+1975] content::RenderViewImpl::OnMessageReceived [0x12C36F81+6561] IPC::MessageRouter::RouteMessage [0x0D90088A+58] content::ChildThreadImpl::ChildThreadMessageRouter::RouteMessage [0x109E2FF6+22] IPC::MessageRouter::OnMessageReceived [0x0D9005DA+58] content::ChildThreadImpl::OnMessageReceived [0x109E228F+1439] IPC::ChannelProxy::Context::OnDispatchMessage [0x0D8AD953+115] base::internal::FunctorTraits<void (__thiscall IPC::ChannelProxy::Context::*)(IPC::Message const &),void>::Invoke<scoped_refptr<IPC::ChannelProxy::Context> const &,IPC::Message const &> [0x0D8A6995+37] base::internal::InvokeHelper<0,void>::MakeItSo<void (__thiscall IPC::ChannelProxy::Context::*const &)(IPC::Message const &),scoped_refptr<IPC::ChannelProxy::Context> const &,IPC::Message const &> [0x0D8A7055+53] base::internal::Invoker<base::internal::BindState<void (__thiscall IPC::ChannelProxy::Context::*)(IPC::Message const &),scoped_refptr<IPC::ChannelProxy::Context>,IPC::Message>,void __cdecl(void)>::RunImpl<void (__thiscall IPC::ChannelProxy::Context::*cons [0x0D8A7453+83] base::internal::Invoker<base::internal::BindState<void (__thiscall IPC::ChannelProxy::Context::*)(IPC::Message const &),scoped_refptr<IPC::ChannelProxy::Context>,IPC::Message>,void __cdecl(void)>::Run [0x0D8AE364+36] base::internal::RunMixin<base::Callback<void __cdecl(void),0,0> >::Run [0x1008A944+68] base::debug::TaskAnnotator::RunTask [0x1008AADF+367] blink::scheduler::TaskQueueManager::ProcessTaskFromWorkQueue [0x191CE676+1094] blink::scheduler::TaskQueueManager::DoWork [0x191CCDA1+737] base::internal::FunctorTraits<void (__thiscall blink::scheduler::TaskQueueManager::*)(base::TimeTicks,bool),void>::Invoke<base::WeakPtr<blink::scheduler::TaskQueueManager> const &,base::TimeTicks const &,bool const &> [0x191C28ED+61] base::internal::InvokeHelper<1,void>::MakeItSo<void (__thiscall blink::scheduler::TaskQueueManager::*const &)(base::TimeTicks,bool),base::WeakPtr<blink::scheduler::TaskQueueManager> const &,base::TimeTicks const &,bool const &> [0x191C2A43+83] base::internal::Invoker<base::internal::BindState<void (__thiscall blink::scheduler::TaskQueueManager::*)(base::TimeTicks,bool),base::WeakPtr<blink::scheduler::TaskQueueManager>,base::TimeTicks,bool>,void __cdecl(void)>::RunImpl<void (__thiscall blink::sc [0x191C2B32+114] base::internal::Invoker<base::internal::BindState<void (__thiscall blink::scheduler::TaskQueueManager::*)(base::TimeTicks,bool),base::WeakPtr<blink::scheduler::TaskQueueManager>,base::TimeTicks,bool>,void __cdecl(void)>::Run [0x191CEFC4+36] base::internal::RunMixin<base::Callback<void __cdecl(void),0,0> >::Run [0x1008A944+68] base::debug::TaskAnnotator::RunTask [0x1008AADF+367] base::MessageLoop::RunTask [0x10105444+660] base::MessageLoop::DeferOrRunPendingTask [0x1010323C+44] base::MessageLoop::DoWork [0x10103822+242] base::MessagePumpDefault::Run [0x1010B4FE+190] base::MessageLoop::RunHandler [0x1010513C+444] base::RunLoop::Run [0x101B7594+52] content::RendererMain [0x12C789FE+830] content::RunNamedProcessTypeMain [0x130B0007+135] content::ContentMainRunnerImpl::Run [0x130AFEC8+488] content::ContentMain [0x130ADDE4+100] ChromeMain [0x02315ED8+264] Original issue's description: > Updating the CLD3 commit hash in DEPS > > The main change are: > - a retrained model with additional script-based features > - replacing "vector" with "std::vector" > > Link to the page in GitHub showing the diff: > https://github.com/google/cld3/commit/fa5974a4d3b5e7934fcb166ff26ed6bfce68b18a > > BUG= 670742 > > Committed: https://crrev.com/f31c5cafc3b085a1f08ed917e518dcc1b30e175f > Cr-Commit-Position: refs/heads/master@{#436034} TBR=groby@chromium.org,rouslan@chromium.org,abakalov@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 670742 Review-Url: https://codereview.chromium.org/2547833004 Cr-Commit-Position: refs/heads/master@{#436196} [modify] https://crrev.com/8377083c34ac86e133e527bacc2ae49a0ead53b5/DEPS
,
Dec 4 2016
,
Dec 4 2016
,
Dec 7 2016
I am going to create a new bug for the most recent version of the code, so I am marking this as "WontFix". |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, Dec 2 2016