You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@echarts.apache.org by GitBox <gi...@apache.org> on 2021/08/04 19:04:41 UTC

[GitHub] [echarts] kulov opened a new issue #15480: Memory leak leading to browser crash

kulov opened a new issue #15480:
URL: https://github.com/apache/echarts/issues/15480


   ### Version
   5.1.2
   
   ### Steps to reproduce
   Created 40 charts in vue.js SPA each chart consisting of 600 data points.
   After period of 1:30h browser crashes with the error below.
   https://snipboard.io/rT5jSf.jpg
   
   Before the crash browser is reporting 4GB of RAM usage.
   https://snipboard.io/U0WFvZ.jpg
   
   Below is code excerpt that is used to chart data.
   
   '
         this.chartCatgories.map((cc) => {
           cc.metrics.map((mtr) => {
             const metricsById = newMetrics.filter((nm) => {
               // check if the metric has the related ID, the related Endpoint and selected location.
               return (
                 nm.id == mtr.text && nm.e == cc.endpoint
                 // && cc.locationFilter.indexOf(nm.d) > -1
               );
             });
   '
   
   Credentials to publicly accessible environment can be provided.
   
   ### What is expected?
   stable and running browser
   
   ### What is actually happening?
   browser is crashing
   
   <!-- This issue is generated by echarts-issue-helper. DO NOT REMOVE -->
   <!-- This issue is in English. DO NOT REMOVE -->


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] kulov commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
kulov commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-892954533






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] kulov commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
kulov commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-893821943


   Dear Yi,
   
   I can provide credentials to the production system and parts of the source code in a private message.
   
   We are now looking at the opportunity to improve the instantiation of echarts within vue.
   Please see existing code excerpt below:
   
   `
         <ui-grid
           class="chart-category align-items-center"
           v-for="category in chartCategories"
           :key="category.endpoint"
         >
   `
   and setting data using
   
   `
         this.chartCategories.map((cc) => {
           cc.metrics.map((mtr) => {
   ...
             mtr.oneData = graphData;
           });
         });
   `
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] pissang commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
pissang commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-893141950


   @kulov Hi, do you mind providing a simple demo that can reproduce the memory leaking issue?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] echarts-bot[bot] commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
echarts-bot[bot] commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-892900329


   Hi! We've received your issue and please be patient to get responded. 🎉
   The average response time is expected to be within one day for weekdays.
   
   In the meanwhile, please make sure that it contains **a minimum reproducible demo** and necessary **images** to illustrate. Otherwise, our committers will ask you to do so.
   
   *A minimum reproducible demo* should contain as little data and components as possible but can still illustrate your problem. This is the best way for us to reproduce it and solve the problem faster.
   
   You may also check out the [API](http://echarts.apache.org/api.html) and [chart option](http://echarts.apache.org/option.html) to get the answer.
   
   If you don't get helped for a long time (over a week) or have an urgent question to ask, you may also send an email to dev@echarts.apache.org. Please attach the issue link if it's a technical question.
   
   If you are interested in the project, you may also subscribe our [mailing list](https://echarts.apache.org/en/maillist.html).
   
   Have a nice day! 🍵


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] pissang commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
pissang commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-893141950


   @kulov Hi, do you mind providing a simple demo that can reproduce the memory leaking issue?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] kulov commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
kulov commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-892954533


   Here are some F12 Allocation profiler results:
   
   ![image](https://user-images.githubusercontent.com/939355/128250170-e03469fe-847b-4ce7-916d-cb2554c591c1.png)
   
   ![image](https://user-images.githubusercontent.com/939355/128250414-9255c4f2-dfd0-48fa-9da4-65bb169ef26a.png)
   
   ![image](https://user-images.githubusercontent.com/939355/128250496-54ad1880-d6d6-45f9-a914-9b19d031ade7.png)
   
   ![image](https://user-images.githubusercontent.com/939355/128250571-0c6917ab-f5e9-4918-aad7-016a3aaa4a69.png)
   
   ![image](https://user-images.githubusercontent.com/939355/128250704-9ed5ed82-6023-4d7b-b92d-982e23f5a213.png)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] kulov commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
kulov commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-892961563


   ![image](https://user-images.githubusercontent.com/939355/128252118-80119cf0-7fea-46e8-a8b7-550bb9b171c2.png)
   
   Data bound to echart is json array. 
   What is the best way to bind data to echart and then release the json array?
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] kulov commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
kulov commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-894858380


   Issue was resolved by fixing the way vue.js instantiates echart objects.
   The original implementation seemed to be instantiating echarts objects on each data bind.
   New implementation is working with object references and setOptions.
   Memory and CPU are now stable.
   ![image](https://user-images.githubusercontent.com/939355/128646285-ac9e3a38-ac91-4955-97fe-af52c39a11f7.png)
   
   Thank you!
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] echarts-bot[bot] commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
echarts-bot[bot] commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-892900329


   Hi! We've received your issue and please be patient to get responded. 🎉
   The average response time is expected to be within one day for weekdays.
   
   In the meanwhile, please make sure that it contains **a minimum reproducible demo** and necessary **images** to illustrate. Otherwise, our committers will ask you to do so.
   
   *A minimum reproducible demo* should contain as little data and components as possible but can still illustrate your problem. This is the best way for us to reproduce it and solve the problem faster.
   
   You may also check out the [API](http://echarts.apache.org/api.html) and [chart option](http://echarts.apache.org/option.html) to get the answer.
   
   If you don't get helped for a long time (over a week) or have an urgent question to ask, you may also send an email to dev@echarts.apache.org. Please attach the issue link if it's a technical question.
   
   If you are interested in the project, you may also subscribe our [mailing list](https://echarts.apache.org/en/maillist.html).
   
   Have a nice day! 🍵


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] Ovilia commented on issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
Ovilia commented on issue #15480:
URL: https://github.com/apache/echarts/issues/15480#issuecomment-895005088


   Glad you found the solution. I'm going to close this. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org


[GitHub] [echarts] Ovilia closed issue #15480: Memory leak leading to browser crash

Posted by GitBox <gi...@apache.org>.
Ovilia closed issue #15480:
URL: https://github.com/apache/echarts/issues/15480


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@echarts.apache.org
For additional commands, e-mail: commits-help@echarts.apache.org