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.

Sign up for GitHub

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

Jump to bottom

V0.11.2.x Performance and UI Fixes Pack #177

Merged
merged 11 commits into from
Feb 12, 2015
Merged

V0.11.2.x Performance and UI Fixes Pack #177

merged 11 commits into from
Feb 12, 2015

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Feb 12, 2015

Performance related fixes:

  • cherry-pick bitcoin commits to solve notifications performance issues during rescan by using notification queue (also resolved conflicts / wiped yet unused functionality to make it work)
  • cherry-pick bitcoin Qt5 OSX QProgressBar CPU-load issue workaround
  • do not calculate anonymized balance while in lite mode
  • solve UI performance problems by caching calculated DS rounds
  • fix progress calculation where denominatedBalance was calculated twice while there was no need for that
  • fix wallet locks in balance / clean up balances calculations a bit

Non-performance but "nice to have" fixes:

  • add "out of sync" message to DS section too
  • fix time to switch "out of sync" message on/of to make UI more consistent (make it lower as we are 4x times faster than bitcoin)
  • IsInitialBlockDownload time range down to 6 hours = ~144 blocks -> 2 x forks detection time (for the same reason as above, also fixing comments there)
  • fix probable security issue with inappropriate rounds chain selection by selecting the shortest one

@UdjinM6
cherry-pick laanwj@023e63d
24e5470
@UdjinM6
resolve conflicts / clean yet unused functionality
f9e8069
@UdjinM6
cherry-pick:
4e94fe1 
[Qt, OSX] QProgressBar CPU-Issue workaround
jonasschnelli@6093aa1
jonasschnelli@0ceab00
[Qt, OSX] fix Qt4.8 compatibility with QProgressBar issue
bitcoin@c5a2282
@UdjinM6
do not calculate anonymized balance during initial download or while …
a948de8 
…in lite mode
@UdjinM6
IsInitialBlockDownload time range down to 6 hours = ~144 blocks -> 2 …
6dc44bd 
…x forks detection time (fixing comments there too)
@UdjinM6
wallet - fix locks / cleanup a bit
90b2a44
@UdjinM6
add "out of sync" message to DS section too
843dc81
@UdjinM6
fix twice denominatedBalance calculations
5011bc2
@UdjinM6
Solve performance problems by caching and fix probable security issue
9823b9e 
- add number of rounds to outputs
- cache rounds calculation results of GetInputDarksendRounds
- select shortest rounds chain instead of the first found
@UdjinM6
allow getAnonymizedBalance calculation on sync again
68cbba1
@UdjinM6
fix time to switch "out of sync" message on/of to make UI more consis…
cc9eef0 
…tent (as we are 4x times faser than bitcoin)
eduffield222 pushed a commit that referenced this pull request Feb 12, 2015
Merge pull request #177 from UdjinM6/v0.11.1.x_fix_gui_performance_is…
a9ff561 
…sues

V0.11.2.x Performance and UI Fixes Pack
@eduffield222 eduffield222 merged commit a9ff561 into dashpay:v0.11.2.x Feb 12, 2015
@UdjinM6 UdjinM6 deleted the v0.11.1.x_fix_gui_performance_issues branch February 13, 2015 22:34
FornaxA pushed a commit to ioncoincore/ion that referenced this pull request Jul 6, 2020
@ioncoincore
Merge pull request dashpay#177 from ckti-ioncore-current/pgp-key
5a51307 
Change public key for ckti
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@UdjinM6 @eduffield222