Syrius: realtime graph not rendering a graph

Archived
1

Problem

Realtime stats are calculated on last week account blocks. Account blocks are retrieved by account block height. The momentum height is calculated using the frontier momentum height minus a fixed momentums per week constant of 60480. The calculated momentum height cannot be used to retrieve the last week account blocks and will always return an empty result.

Solution

Page through the account blocks until the momentum height of the confirmation detail is smaller than or equal to the calculated momentum height.

Implementation: https://github.com/KingGorrin/syrius/tree/fix-rt-stats

Questions

  1. What should be the page size. The infinite scroll bloc uses a page size of 10.
1 Like
2

That value tends to be 10.
image
I presume it’s to limit the amount of data that the node needs to pull and send back to the client.
rpcMaxPageSize = 1024

I haven’t experimented with this value much. Are you concerned 10 is too much or not enough?

3

No not too much, but too little maybe.

I’ve uploaded a fix for the impl. here https://github.com/KingGorrin/syrius/tree/fix-rt-stats

The less requests made the better. Ideally it should be avarage user tx a day times 7, which is dynamic depending the network usage with a minimum of 10.

But for now, 10 is fine.

1 Like
4

We can always adjust these variables. I’ll be testing this with mainnet and devnet to see if it works well/impacts dashboard performance.

5

Worked well on mainnet and devnet. I reviewed the code changes and didn’t identify any issues.

I think this could slide into v0.0.6 if others are okay with that.
We may need a public discussion about what is included in that version and what can wait until the next version (aka “negotiations”).

@aliencoder

2 Likes
6

I agree with you @sol

I think we only lack Linux and Windows packaging at the moment. After that we’re good to go and submit the PR.

2 Likes
7

Which will never get merged… but I’ll be the first to download it… I really appreciate all your efforts zirs.

1 Like
8

oof. I’ll be very sad if that’s what happens.
Mr Kaine promised us that he will merge changes.

1 Like
9

We need it merged under the official release imo, we can all try help get traction on that if needed

10

Too much positive traction for it to be ignored.

1 Like
11

OP correction.

What?

The realtime statistics are never calculated.

Why?

The realtime statistics are calculated on last week account blocks. Account blocks are retrieved by account block height. The momentum height is calculated using the frontier momentum height minus a fixed momentums per week constant of 60480. The calculated momentum height cannot be used to retrieve the last week account blocks and will always return an empty result.

How?

Page through the account blocks until the momentum height of the confirmation detail is smaller than or equal to the calculated momentum height.

Anything else?

  1. The paging uses a static page size of 10.

Implementation

https://github.com/KingGorrin/syrius/tree/fix-rt-stats

1 Like