New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 670742 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 670984



Sign in to add a comment

Merging a CL about language identification into M56

Project Member Reported by abakalov@chromium.org, Dec 2 2016

Issue description

We'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.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f31c5cafc3b085a1f08ed917e518dcc1b30e175f

commit f31c5cafc3b085a1f08ed917e518dcc1b30e175f
Author: abakalov <abakalov@chromium.org>
Date: Fri Dec 02 21:30:41 2016

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 

Review-Url: https://codereview.chromium.org/2546943002
Cr-Commit-Position: refs/heads/master@{#436034}

[modify] https://crrev.com/f31c5cafc3b085a1f08ed917e518dcc1b30e175f/DEPS

Cc: groby@chromium.org djweiss@chromium.org zkoch@chromium.org riesa@chromium.org
Labels: Merge-Request-56

Comment 3 by dimu@chromium.org, Dec 3 2016

Labels: -Merge-Request-56 Merge-Review-56 Hotlist-Merge-Review
[Automated comment] DEPS changes referenced in bugdroid comments, needs manual review.
Project Member

Comment 4 by bugdroid1@chromium.org, 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

Comment 5 by kbr@chromium.org, Dec 4 2016

Blockedon: 670984
Labels: -Hotlist-Merge-Review -Merge-Review-56
Status: WontFix (was: Untriaged)
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