-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
Improve network monitor #1166
Comments
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
I've spent some time working on these:
The problem this intended to solve
Fixed would be:
so if is type_A and doesnt work well with parser A will immediately raise that error and will not be wasting time. right now there are just 2 parsers, PeerChanges and ContractChanges. Is it worth it to spend time on this? |
This happens on the network_metrics_server (backend) on on the frontend is on |
I think it makes sense if we are going to keep making changes or adding more message types in the future, I am in favour of doing the change as it shouldn't be that bad and will clean up code |
Understood, will do it! |
…reenet#1166-improve-network-monitor
…reenet#1166-improve-network-monitor
…reenet#1166-improve-network-monitor
Co-authored-by: nacho.d.g <[email protected]>
…reenet#1166-improve-network-monitor
…reenet#1166-improve-network-monitor
…reenet#1166-improve-network-monitor
It'll be sending more data to network monitor. Creating more types and displaying it better in the web app.
There was a PR associated with this task. Im continuing with it.
#1052
Frontend - Typescript
transactions-data.ts
any
to actual types.mocks.ts
type_definitions.ts
Backend
Do we need these?
added on 29/08
Contract Ring
Transaction Tree
Main Nodes Ring Enhancement
sketch https://excalidraw.com/#json=g2UXiuA2T-P3gg9LY1JdR,Pjg-xsQFuxMNQGGphbxAKA
You'll we able to see peers and/or contracts at the same time.
The color of the big circle will be the way to indicate the density of internal connections. Red, yellow and green for low, medium and high.
Ideally a peer nucleus will be very close to a contract nucleus. And should have similar size.
The text was updated successfully, but these errors were encountered: