Skip to content
This repository has been archived by the owner on Jan 6, 2022. It is now read-only.

Show more network stats in the UI #248

Closed
1 task done
yoshuawuyts opened this issue Feb 20, 2017 · 11 comments
Closed
1 task done

Show more network stats in the UI #248

yoshuawuyts opened this issue Feb 20, 2017 · 11 comments

Comments

@yoshuawuyts
Copy link
Contributor

yoshuawuyts commented Feb 20, 2017

When downloading a dat, it is often useful to know things about the network. Currently we're tracking the peer count, but it'd also be good if we:

  • track upload / download speeds
    - track "peers that have the full dataset" vs "peers that have a partial replica"

Labeling this as high priority because not having any information about the state of the network can be frustrating for users as they've got no idea what's causing slowness or even halted downloads.


@Kriesse any ideas on a UI for this?

@yoshuawuyts
Copy link
Contributor Author

As per #23 (comment) we can implement these through dat.stats.network.uploadSpeed and dat.stats.network.downloadSpeed. There are no events so currently they're updated using a loop

@yoshuawuyts yoshuawuyts modified the milestone: Release 2017-02-28 Feb 20, 2017
@Kriesse
Copy link
Collaborator

Kriesse commented Feb 21, 2017

@yoshuawuyts Updated mockup. Please review, does that make sense?
image

Related, as it also helps to the user to understand what the status is of each Dat: #184

Do we need stats on peers with partial/full datasets yet? In favor of keeping the overview uncluttered, suggest to have this part of a potential future feature: extended stats (e.g. in a panel on the bottom, think utorrent).

@okdistribute
Copy link
Collaborator

okdistribute commented Feb 21, 2017

Yeah, I like it!

Let's chat about a desktop roadmap for the next milestone with @maxogden, it'd be good to get a sense of what would be the most exciting to show off in a demo. It could be easy to rewrite https://github.com/mafintosh/dat-drop here in a panel when you click a dat.

@okdistribute
Copy link
Collaborator

And yes, peer partial replication stats should be a separate issue.

@yoshuawuyts yoshuawuyts mentioned this issue Feb 21, 2017
2 tasks
@yoshuawuyts
Copy link
Contributor Author

@Kriesse neat, yeah that makes a lot of sense! Digging it.

And yeah sounds good; created a separate issue for the detail view #260

@yoshuawuyts
Copy link
Contributor Author

@Karissa gonna check it out - hadn't seen that yet ✨

@juliangruber
Copy link
Collaborator

getting started with this now

@juliangruber
Copy link
Collaborator

started my work here: #268

@Kriesse
Copy link
Collaborator

Kriesse commented Feb 23, 2017

@juliangruber Oh perfect thanks for preparing that! Will take it from here and start with styling.

@Kriesse Kriesse assigned juliangruber and unassigned Kriesse Feb 23, 2017
@Kriesse
Copy link
Collaborator

Kriesse commented Feb 23, 2017

@juliangruber styling done for now! i'll assign to you, let me know if i missed anything.

@juliangruber
Copy link
Collaborator

merged!

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

No branches or pull requests

4 participants