Issue metadata
Sign in to add a comment
|
Poor performance since last update
Reported by
mothersh...@outlook.com,
Aug 3
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.84 Safari/537.36 Steps to reproduce the problem: 1. not sure how to replicate 2. 3. What is the expected behavior? while developing webgl content with the THREE.js API I didn't run in to any noticeable performance issues. However, since the last Chrome update my project runs a significantly reduced performance. What went wrong? After the contents of the page load animation frames now take in execess of 100ms to process where as before the latest Chrome update they only took ~9ms Did this work before? N/A Chrome version: 68.0.3440.84 Channel: stable OS Version: Flash Version: The two screen shots are showing performance of the latest version of Chrome and the one previous. 67.#### If there is any additional information I can provide please let me know.
,
Aug 3
Unfortunately I can't use Chrome Canary since my project is being built within Linux. the contents of the page are supplied by a server. I could just run it live. Seems a lot of trouble though to set it all up. The previous version of Chrome that worked was the one right before this latest one. 67.0.3396.79 One additional side not. May help weed out the source of the problem. Previously on previous Chrome builds the page would load, and then after 30 seconds the performance would drop significantly. I'd refresh the page and then the content would load back up and the problem would be gone for good. Now the problem happens regardless of the number of times I refresh the page.
,
Aug 3
This seems to be the very function that is causing the bottleneck. Any ideas why?
copyVector3sArray:
function(a)
{
for(var b=this.array,c=0,d=0,e=a.length;d<e;d++)
{
var f=a[d];
void 0===f&&(console.warn("THREE.BufferAttribute.copyVector3sArray(): vector is undefined",d),f=new n);
b[c++]=f.x;
b[c++]=f.y;
b[c++]=f.z
}
return this
}
,
Aug 3
the size of the array it is copying is 468000 in length
,
Aug 3
Without a demo there's nothing for the Chromium team to go on, I'm afraid. Try bisecting yourself using the link I gave in comment 1.
,
Aug 3
I thought bisecting only determines which version the change occurs in? Well, here's another screen shot this.length is 468000, and a.length is 156000.
,
Aug 3
Are there perhaps new memory limitations imposed now? I know my project uses up a lot of RAM, could that be causing issues for these particular arrays that are being copied?
,
Aug 3
Bisecting produces a list of commits (usually 1-20) so it's usually obvious which change caused the regression. This is a much more reliable way to determine the reason compared to guessing. I don't think Chromium team would guess without being able to reproduce the problem as there are lots of cases where the reported problems are caused by something else.
,
Aug 6
,
Aug 8
reporter@ - Thanks for filing the issue...!! Could you please provide a sample test file/url to test the issue from TE-end. This will help us in triaging the issue further. Thanks...!!
,
Aug 18
,
Aug 27
Closing bug because it can't be reproduced. Please reopen if you are able to provide a reproducible test case :)
,
Aug 27
The NextAction date has arrived: 2018-08-27 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by woxxom@gmail.com
, Aug 3