Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

UI causes Parity to consume too much CPU #3240

Closed
jacogr opened this issue Nov 7, 2016 · 1 comment · Fixed by #3605
Closed

UI causes Parity to consume too much CPU #3240

jacogr opened this issue Nov 7, 2016 · 1 comment · Fixed by #3605
Assignees
@ngotchac
Labels
F2-bug 🐞 The client fails to follow expected behavior.
Projects
User Interface
Milestone
1.5 Nativity

Comments

@jacogr
Copy link
Contributor

jacogr commented Nov 7, 2016

Both Parity & Chrome shows higher than normal CPU usage when the app is open. Check -

  • status optimisations (slower, not all)
  • running timers, polling
@jacogr jacogr added F2-bug 🐞 The client fails to follow expected behavior. M6-ui labels Nov 7, 2016
@jacogr jacogr added this to the 1.4 Civility milestone Nov 7, 2016
@gavofyork gavofyork modified the milestones: 1.5 Tenuity, 1.4 Civility Nov 7, 2016
@ngotchac
Copy link
Contributor

Some work have been done in #3227 to fix these. Will make a smaller PR from this one.

@ngotchac ngotchac self-assigned this Nov 17, 2016
ngotchac added a commit that referenced this issue Nov 17, 2016
@ngotchac
Smarter Status Redux dispatch #3240
50473a6
ngotchac added a commit that referenced this issue Nov 17, 2016
@ngotchac
Smarter pinging. Don't waster resources #3240
5fd66d2
ngotchac added a commit that referenced this issue Nov 17, 2016
@ngotchac
Smarter Polling #3240
1c3923c 
  Only refresh statuses when on Status Page for
  some statuses (eg. hashrate)
@ngotchac ngotchac mentioned this issue Nov 18, 2016
Merged
ngotchac added a commit that referenced this issue Nov 18, 2016
@ngotchac
Don't auto-subscribe for contracts #3240
0e53f7d
ngotchac added a commit that referenced this issue Nov 18, 2016
@ngotchac
Smarter Balance : don't re-instantiate contracts, fetch tokens #3240
601e9ec
ngotchac added a commit that referenced this issue Nov 19, 2016
@ngotchac
Smarter Balance Tokens fetching (update image when needed) #3240
9ebe84a
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
Attaching to TokenReg Events instead of fetching for each block #3240
3f332a0
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
Unsubscribe from shapeshit... #3240
7db0ad1
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
Unsubscribe from EthFilter if used once (resolved) #3240
2ec182a
@ngotchac ngotchac mentioned this issue Nov 21, 2016
Merged
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
React Perf in dev mode // Smarter background component #3240
ca5fd0b
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
Better TabBar #3240
a317e4f
ngotchac added a commit that referenced this issue Nov 21, 2016
@ngotchac
Progressive Accounts List Rendering #3240
35fdbf3
@ngotchac ngotchac mentioned this issue Nov 21, 2016
Merged
jacogr pushed a commit that referenced this issue Nov 23, 2016
@ngotchac @jacogr
Smarter Tokens fetching (#3546)
33dd491 
* Don't auto-subscribe for contracts #3240

* Smarter Balance : don't re-instantiate contracts, fetch tokens #3240

* Smarter Balance Tokens fetching (update image when needed) #3240

* Attaching to TokenReg Events instead of fetching for each block #3240

* Unsubscribe from shapeshit... #3240

* Unsubscribe from EthFilter if used once (resolved) #3240

* Remove warning

* PR review fix

* Typo

* Better contract JS API : one subscribe for all #3546

* Fixed test
@ngotchac ngotchac mentioned this issue Nov 25, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ngotchac ngotchac

Labels
F2-bug 🐞 The client fails to follow expected behavior.
Projects
No open projects
User Interface
Done (1.5.0)
Milestone
1.5 Nativity
Development

Successfully merging a pull request may close this issue.

Smarter balance fetching openethereum/parity-ethereum
3 participants
@gavofyork @jacogr @ngotchac