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/10/11 22:48:15 UTC
[GitHub] [echarts] ChristianRemwood opened a new issue #15861: on('click) events don't work with rapid data updates.
ChristianRemwood opened a new issue #15861:
URL: https://github.com/apache/echarts/issues/15861
### Version
5.2.1
### Steps to reproduce
Head to this example (https://echarts.apache.org/examples/en/editor.html?c=dynamic-data)
Change the update time from 2100 to 21
Attempt to click "Dynamic Bar" Legend text
### What is expected?
The "Dynamic Bar" data should be toggled
### What is actually happening?
There is a high likelihood that the data will not be toggled unless your mouse-down and mouse-up events happen under 21ms
---
I have temporarily fixed this on a local copy by changing the event type for "dispatchSelectAction" from click to mouseup. This is not ideal, but is enough for what I needed short term. ex: ( LegendView.ts: itemGroup.on('mouseup', curry(dispatchSelectAction, name, null, api, excludeSeriesId)) )
I have also discovered the issue seems to be with the generated javascript section "if(this._downEl!==this._upEl||!this._downPoint||Dt(this._downPoint,[e.zrX,e.zrY])>4)" in min.js.
Specifically the section 'this._downEl!==this._upEl'
After expanding the objects I can see they look nearly identical, but the parent sections are drastically different. I suspect the parent ID changes each time setOption() is called.
<!-- 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] ChristianRemwood commented on issue #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
ChristianRemwood commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-941896474
I am feeding the charts live data from several different sensors and the
data needs to refresh every 10-20ms in order for us to not miss anything.
The refresh rate seems to be no problem for most things, but the on click
events don't work. The reason I need the on click elements to work in my
use case is we need to toggle on and off several data sets to get a better
picture of what's happening at all stages of our sensors processing.
On Tue, Oct 12, 2021, 7:15 PM Wenli Zhang ***@***.***> wrote:
> May I ask in which scenio do you have this requirement? Do the data change
> as frequent as such?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/echarts/issues/15861#issuecomment-941849805>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AILLEFQUVYOXIWH3Q5D2YHTUGTTTNANCNFSM5FZGPSNA>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
--
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 #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
Ovilia commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-941849805
May I ask in which scenio do you have this requirement? Do the data change as frequent as such?
--
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] ChristianRemwood commented on issue #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
ChristianRemwood commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-944348391
I did also observe that other mouse events trigger no problem. It's just
the click events. Also the click events do step down through the code, but
they return early at the line of code I mentioned previously.
On Thu, Oct 14, 2021 at 8:50 PM Wenli Zhang ***@***.***>
wrote:
> Hi, refreshing everying 10-20ms seems to be too frequent in most cases.
> Maybe you should first consider if you do need such frequent updates. The
> animation usually takes 500ms-1000ms to complete, so if you do update so
> frequently, you should consider switching off the animation by
> 'option.animation = false'.
> That being said, we should still look into the reason why the event is not
> triggered.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/echarts/issues/15861#issuecomment-943942769>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AILLEFSBG4IQZLFJHSQQDJTUG6JGVANCNFSM5FZGPSNA>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
--
-Christian Remwood
--
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 #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
echarts-bot[bot] commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-940499012
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 to 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 #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
Ovilia commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-941849805
May I ask in which scenio do you have this requirement? Do the data change as frequent as such?
--
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] ChristianRemwood commented on issue #15861: on('click) events don't work with rapid data updates.
Posted by GitBox <gi...@apache.org>.
ChristianRemwood commented on issue #15861:
URL: https://github.com/apache/echarts/issues/15861#issuecomment-941896474
I am feeding the charts live data from several different sensors and the
data needs to refresh every 10-20ms in order for us to not miss anything.
The refresh rate seems to be no problem for most things, but the on click
events don't work. The reason I need the on click elements to work in my
use case is we need to toggle on and off several data sets to get a better
picture of what's happening at all stages of our sensors processing.
On Tue, Oct 12, 2021, 7:15 PM Wenli Zhang ***@***.***> wrote:
> May I ask in which scenio do you have this requirement? Do the data change
> as frequent as such?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/echarts/issues/15861#issuecomment-941849805>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AILLEFQUVYOXIWH3Q5D2YHTUGTTTNANCNFSM5FZGPSNA>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
--
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