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

Uptime #190

Merged
merged 33 commits into from
Jan 26, 2023
Merged

Uptime #190

merged 33 commits into from
Jan 26, 2023

Conversation

dskvr
Copy link
Collaborator

@dskvr dskvr commented Jan 10, 2023

This is experimental.

Resolves #100
Resolves #148

Screen Shot 2023-01-11 at 12 51 54 AM

TODO:

  1. normalize height based on latency. probably inverted from screenshot, short dark green = good, tall deep red = bad
  2. normalize color based on latency, green->orange->red
  3. Calculate average latency in range.
  4. Compress the data so that it can be cached, otherwise it will break everything.
  5. Subscribe to the heartbeat to update realtime.
  6. Catch up on pageload
  7. Truncate old data.
  8. Display on relay list

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for nostrwatch ready!

Name Link
🔨 Latest commit 42f2d89
🔍 Latest deploy log https://app.netlify.com/sites/nostrwatch/deploys/63d2daaa2e63cf0008dcfa18
😎 Deploy Preview https://deploy-preview-190--nostrwatch.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@dskvr dskvr marked this pull request as draft January 22, 2023 15:21
@dskvr dskvr marked this pull request as ready for review January 26, 2023 18:00
@dskvr
Copy link
Collaborator Author

dskvr commented Jan 26, 2023

@cameri If you have a moment, please review at deploy link.

@dskvr
Copy link
Collaborator Author

dskvr commented Jan 26, 2023

... also, should ixed a production bug that's been online for the last week or so. Would have only affectetd new users, unfortunately.

@cameri
Copy link
Contributor

cameri commented Jan 26, 2023

@cameri If you have a moment, please review at deploy link.

Uptime not showing on the relay list table.

@dskvr
Copy link
Collaborator Author

dskvr commented Jan 26, 2023

Thank you, just noticed it as well, there's a referencing issue. You can force it by going to a single relay and then going back, if you would like to review that.

@dskvr
Copy link
Collaborator Author

dskvr commented Jan 26, 2023

@cameri Resolved.

@dskvr dskvr merged commit 2daa902 into develop Jan 26, 2023
@dskvr dskvr deleted the feature/heartbeat branch June 21, 2023 14:28
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.

Feature request: Relay uptime Feature Request: Average Read Latency
2 participants