Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-select: high cpu and unresponsive ui #7706

Open
blackboxlogic opened this issue Jun 15, 2020 · 2 comments
Open

Multi-select: high cpu and unresponsive ui #7706

blackboxlogic opened this issue Jun 15, 2020 · 2 comments
Labels
performance Optimizing for speed and efficiency

Comments

@blackboxlogic
Copy link
Contributor

In the dev branch of ID 2.17.3 (currently here) where multi-select has been implemented, when I select many (25) nodes, my cpu sits at 60% and click events in iD start to "miss", meaning that the user actions occasionally don't illicit any response from the UI.

I suspect that the pulsing red boarder on selected elements is expensive. Consider:

  • Limit the quantity of elements which can be selected simultaneously
  • Reduce the cost of the pulsing selection-highlighter
  • Add a setting for the selection-highlighter to be static (non-animated)
  • Make the selection-highlighter static when multiple elements are selected
@quincylvania
Copy link
Collaborator

quincylvania commented Jun 15, 2020

The good news is: this used to be a lot worse! See #3571. On my setup I can select a few hundred nodes and suffer only a minor performance hit. But I'm concerned that you're seeing something worse. What browser/system are you testing with?

#2911 #3919 was also about improving the selection animation performance but was closed as stale.

@quincylvania quincylvania added performance Optimizing for speed and efficiency waitfor-info Waiting for more info labels Jun 15, 2020
@blackboxlogic
Copy link
Contributor Author

blackboxlogic commented Jun 15, 2020

Windows 10 home
Intel Core i7-7500 @ 2.8 GHz
16 gigs ram
Firefox 77.0.1 (64-bit)

No errors in the developer console in Firefox. The selection-highlighter also gets jittery and starts skipping transparency frames, jumping from opaque to not.

I just tested on chrome and CPU stays much lower but click events get dropped about the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Optimizing for speed and efficiency
Projects
None yet
Development

No branches or pull requests

2 participants