search

Project-Faster / qpep

A working version of qpep standalone go client/server, designed to accelerate high-latency connections, like those provided by geostationary satellites.
https://docs.projectfaster.org
Other
3 stars 1 forks source link

Simple bandwidth activity meter in the UI #3

Closed bizzbyster closed 2 years ago

bizzbyster commented 2 years ago

Goal: present the user with some high level data to convince them that the QPEP client is actually working

parvit commented 2 years ago

The GUI part is going to take some research time to see how best to implement.

In the mean time, do you have any additional statistics you would like to collect?

parvit commented 2 years ago

@bizzbyster qpep_interface.pdf Here's my proposal for the change, let me know if you have any question for this.

I estimate a 2 week effort at minimum.

bizzbyster commented 2 years ago

@parvit What javascript framework are you using for this? The functionality looks great to me but the aesthetics might be improved. Maybe you check this in as designed and then @nachocardozop we can ask another engineer like @avnerjose to update the look and feel?

parvit commented 2 years ago

Hi i'm sorry for the confusion but that is just a functional mockup i did not intend for it to be the final look.

I was thinking of using a material theme and the Aurelia framework for the actual js.

If you do not have functional objections i will begin working on it.

Il giorno mer 27 lug 2022 alle ore 16:34 bizzbyster < @.***> ha scritto:

@parvit https://github.com/parvit What javascript framework are you using for this? The functionality looks great to me but the aesthetics might be improved. Maybe you check this in as designed and then @nachocardozop https://github.com/nachocardozop we can ask another engineer like @avnerjose https://github.com/avnerjose to update the look and feel?

— Reply to this email directly, view it on GitHub https://github.com/Project-Faster/qpep/issues/3#issuecomment-1196842130, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWGJD4CK2BKXJGPYSFFO7T3VWFCGDANCNFSM53TBH2HA . You are receiving this because you were mentioned.Message ID: @.***>

bizzbyster commented 2 years ago

Sounds good if @mfoxworthy is good with the functionality.

mfoxworthy commented 2 years ago

Sounds good if @mfoxworthy is good with the functionality.

Functionality is perfect. There is no question whether the PEP is working or not. That is the main goal.

parvit commented 2 years ago

I can start already, however to complete it i'll require the list of statistics/performance counters that should be collected.

parvit commented 2 years ago

After today's meeting it was confirmed that the following statistics are required to be collected for clients:

parvit commented 2 years ago

Hello everyone, the development is completed in the PR #18, please test and review.

parvit commented 2 years ago

Hi, any feedback for this? @mfoxworthy @bizzbyster

bizzbyster commented 2 years ago

I will be taking a look today-- sorry for my slowness. I believe Mike has been testing it so I'm sure we'll get his review comments soon.

parvit commented 2 years ago

Feature tested positively and merged.