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

[RPC][GUI] Added more progress indicators during wallet start up, also shows via… #702

Merged
merged 1 commit into from
Jun 7, 2020

Conversation

mxaddict
Copy link
Contributor

@mxaddict mxaddict commented Jun 5, 2020

closes #654

I'm sure this will be helpful for people wanting CLI progress during wallet start up, and also makes the splash screen have more feedback to the user.

But not sure how much this slows down the start up as it's basically tracking progress for most of the heavy loops in wallet start up.

@aguycalled do you think we should add this?

It's just something that came to mind when I took a look @ pr #654

@mxaddict mxaddict changed the title Added more progress indicators during wallet start up, also shows via… [RPC][GUI] Added more progress indicators during wallet start up, also shows via… Jun 5, 2020
@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 5, 2020

Ok, I did some testing with this code, the wallet only loads like ~1% to ~2% slower than without this change on my PC, can you guys test and let me know if it performance ok?

@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 5, 2020

Capture2
Capture
Capture3

@Xa5r
Copy link

Xa5r commented Jun 5, 2020

Rescanning (changing the wallet.dat) it also what took a very long time yesterday

@navbuilder
Copy link

A new build of a455624 has completed succesfully!
Binaries available at https://build.nav.community/binaries/added-more-progress-indicators

@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 5, 2020

Rescanning (changing the wallet.dat) it also what took a very long time yesterday

@Xa5r could you test this PR and see if it makes your wallet start time slow?

@Xa5r
Copy link

Xa5r commented Jun 6, 2020

Its difficult to compare because the loading time is always different.
But I noticed that by bootstraping the notice "Loading block guts xxx..." goes over 4.3 million, more blocks than the current highest block.

@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 6, 2020

Its difficult to compare because the loading time is always different.
But I noticed that by bootstraping the notice "Loading block guts xxx..." goes over 4.3 million, more blocks than the current highest block.

I don't think that loading block guts only loads active chains, I think it also loads forked/abandoned chain heads, which might explain why it loads more than the active chain.

I'm just assuming though, might be better to ask @aguycalled about it.

@aguycalled
Copy link
Member

Its difficult to compare because the loading time is always different.
But I noticed that by bootstraping the notice "Loading block guts xxx..." goes over 4.3 million, more blocks than the current highest block.

I don't think that loading block guts only loads active chains, I think it also loads forked/abandoned chain heads, which might explain why it loads more than the active chain.

I'm just assuming though, might be better to ask @aguycalled about it.

yes that's right

@aguycalled
Copy link
Member

tested gitian osx, surprisingly, my wallet in mainnet loads 10 times faster with this branch

@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 7, 2020

tested gitian osx, surprisingly, my wallet in mainnet loads 10 times faster with this branch

I don't know how that would happen, but it's a good thing I guess 🗡️

@mxaddict
Copy link
Contributor Author

mxaddict commented Jun 7, 2020

tested gitian osx, surprisingly, my wallet in mainnet loads 10 times faster with this branch

I don't know how that would happen, but it's a good thing I guess 🗡️

@chasingkirkjufell can you maybe test this branch as well on windows?

I've tested on Ubuntu 18.04 (Which performed about 1-2% slower)

Copy link
Contributor

@chasingkirkjufell chasingkirkjufell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love it. It is about 1% slower on with Gitian build on Windows 10 but having the extra info is really nice.

@mxaddict mxaddict merged commit d6f0acc into navcoin:master Jun 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add progress info to "Loading block index..." RPC message
5 participants