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

Fix block animations while syncing #3651

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mononaut
Copy link
Contributor

@mononaut mononaut commented Apr 6, 2023

This PR fixes some issues with the blockchain animations while the backend is syncing (e.g. after a fresh install, or being offline for a time), and also during bitcoin core initial block download.

IBD

Before:

long-before.mp4

After:

long-after.mp4

Backend syncing

Before:

before.mp4

After:

after.mp4

@cla-bot cla-bot bot added the cla-signed label Apr 6, 2023
@softsimon softsimon added this to the v2.5.1 milestone Apr 7, 2023
@softsimon
Copy link
Member

softsimon commented Apr 8, 2023

A question regarding the "Backend syncing" part. It seems the blocks will now flow from left to right even though the mempool is not matching with null as match rate? Before we just popped them up to the right side and only animate over when there is a match in the mempool and mined blocks.

Here I started the backend after being offline for a few hours and blocks were animating, despite the mempool basically not changing.

Not sure what's right and wrong here.

Screenshot 2023-04-08 at 19 40 44

Screenshot 2023-04-08 at 19 40 47

@softsimon softsimon self-requested a review April 8, 2023 12:26
@mononaut
Copy link
Contributor Author

mononaut commented Apr 10, 2023

Good point, we shouldn't animate the latest mined block in that case. I'll fix it.

Copy link
Member

@softsimon softsimon left a comment

Choose a reason for hiding this comment

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

Just changing state to changes requested.

@softsimon softsimon modified the milestones: v2.5.1, v3.1 Jul 17, 2023
@wiz
Copy link
Member

wiz commented Apr 9, 2024

@knorrium isn't this old PR fixing your thing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants