NDT-DEV email list created

[Tiziana Refice <>]

Hi Sebastian,

On Wed, Feb 19, 2014 at 1:55 PM, Sebastian Kostuch <> wrote:

Hi Tiziana,
Thank you so much for all your reviews.

Thanks to you for taking the time and making the changes!

 

I have fixed almost all of them. The only one left is with copy function.
This is however related to some another question about current state of UI in flash client (see below).

While working on Issue 115 I have encountered some difficulties with dynamically adding
mx or spark controls (to use ProgressBar during tests and Tooltip in mentioned copy function).
It seems like now it is possible to only add basic flash components from flash.display package
due to lack of mxml file which is responsible for containing info about mx and/or spark
components and creating proper container for such objects (which is rather hard to achieve
in pure AS. Also, such an approach is recommended while making little more advanced GUI).

So my question is if we would need such an approach with moving our GUI information to such mxml file?

Are you talking about this file

https://code.google.com/p/ndt/source/browse/trunk/flash-client/ndt-flash-config.xml?

Which information would you be moving to the mxml?

Also, I believe that portability is very important.

This is why I tried to use as pure AS as possible.

Could you specify which clients we would loose if we implemented the changes that you are describing?

It would probably be good when thinking about making some major improvements to UI but if we think
about flash client as background for UI created in HTML and JS then it doesn't seem to be key feature.

This is a question still under discussion.

One one hand is nice to have a self contained client.

One the other hand, having a very basic Flash UI and a fancier UI in JS is definitely a valid alternative.

What's you opinion about this?

Personally, I would like to at least avoid to have in Flash UI features that would affect the accuracy of the client (i.e., all the features that need to be updated while the test is running).

In particular, I am concerned about the progress bar.

I think that, unless we move to Flash multi-thread, the progress bar should not be implemented in AS.

However, this is just my personal intuition.

If you have numbers that show the impact of having such UI feature in AS,  I would be interested in them.

If the answer is positive then it would probably be good to create separated issue related to it and
as a temporary solution what do you think about:
1. Simplifying progress bar to only show percentage of completion as plain text without ProgressBar control
2. Moving copy button at the bottom of text console (which will be shorten a little in height). Would
it be more intuitive?

I don't see a great advantage of having the copy button (without any kind of feedback to the user), compared to just have the user select and copy the text "manually".

The latter seems much more intuitive.

I hope my thoughts are clear. If not, feel free to ask any questions :)

Speaking of UI changes, have you looked at the possibility to add a scroll bar to the result tabs? 

Best,

Tiziana

Kind regards,
Sebastian Kostuch

On 18.02.2014 10:23, Tiziana Refice wrote:

(Adding Meredith, to keep her in the loop of the changes to the Flash client).

HI Sebastian,

I reviewed all the pending changes except for FlashIssue114.

I will look into that in the next few days.

Thank you!

Tiziana

On Mon, Feb 17, 2014 at 11:35 AM, Sebastian Kostuch <> wrote:

Hi Tiziana,
thanks a lot for your reviews. Keeping them in mind I have made some fixes in my code.
You can see changes on related branches, also I list them here:
1. FlashIssue113 - set missing variables in clearResults method and changed position of restart button
    (the problem with tabs instead of white spaces I have fixed right afer first commit, it can be viewed here)
2. FlashIssue118 - changed color of active button
3. FlashIssue116 - changed text color and fixed code formatting

Please let me know if everything looks now ok for you.

Regards,
Sebastian Kostuch

On 14.02.2014 11:50, Tiziana Refice wrote:

Done! I reviewed all the changes.

Please let me know if my comments are not clear.

Thanks!

On Thu, Feb 13, 2014 at 11:32 AM, Tiziana Refice <> wrote:

Hi Sebastian,

FYI: I am planning to review all yr changes to the AS code in the next few days. Most likely on Sat.

Cheers,

Tiziana

On Thu, Feb 13, 2014 at 11:16 AM, Sebastian Kostuch <> wrote:

Hi,
recently I have pushed few commits related to flash client:
1. Added possibility to restart test after first completion (FlashIssue113 branch)
2. Active button in result view is now being highlighted (FlashIssue118 branch)
3. Console output is now little more legible (FlashIssue116 branch)

Please feel free to review them. Any comments are welcome and appreciated.

Regards,
Sebastian Kostuch

On 10.02.2014 19:06, Tiziana Refice wrote:

Hi Will,

I will review the patch in the next few days.

Thanks!

On Sat, Feb 8, 2014 at 6:32 PM, Will Hawkins <> wrote:

On Saturday, February 8, 2014 08:44 EST, Tiziana Refice <> wrote:

> On Fri, Feb 7, 2014 at 2:29 PM, Aaron Brown <> wrote:
>
> >  Hey Sebastian,
> >
> >  On Feb 7, 2014, at 9:03 AM, Sebastian Kostuch <>
> >  wrote:
> >
> > Hi,
> > after comparing flash and java clients I have prepared some initial list
> > of suggestions/improvements which could be done to make the flash one
> > better. Please check them and let me know which one are worth/not worth
> > doing. Also if you have something to add feel free to announce it in
> > response :)
> >
> > Flash client improvements:
> > 1. Add some basic options at start page (for example prefered IP protocol,
> > server name)
> >
> >
> >  I'd prefer these options be hidden in an "advanced" button that would pop
> > up a dialog box allowing you to set them. The groal, of sorts, should be to
> > make the default 'view' simple, and with as little jargon as possible.
> > Basically, just a "Start Test" button, and some text like it is now. Also,
> > it'd be great if the text could be set via the HTML so sites could
> > customize it.
> >
>
> About the protocol, I am not sure whether it's possible to select IPv4 or
> IPv6 from within Flash.
> (Flash gives very limited access to the TCP/IP stack).
>
> Just FYI: About the server selection, I will commit shortly a JS snippet to
> automatically select the closest NDT server (among all the NDT servers
> hosted on the M-Lab platform).
>

Hey Tiziana!

I posted a patch to some of the code related to this: https://code.google.com/p/ndt/issues/detail?id=111 I hope that it works. Let me know if you have feedback!

Will

>
> > 2. Add possibility to re-run test after first completion
> >
> >
> >  Seems reasonable.
> >
>
> SGTM
>
> >
> > 3. On Results/Details/Errors/Debug views add appropriated title (or just
> > highlight somehow active button)
> >
> >
> >  It'd probably be good to make it easy for a site to disable the DEBUG
> > button. Also, ERRORS should probably just be merged into results somehow.
> >
> > 4. Maybe add ability to download results as file
> >
> >
> >  You'd probably need to save the results that are already there since you
> > can't re-request results from NDT; it throws them out after its done with
> > the test.
> >
> > 5. Add some basic progress bar during tests (indicating how many tests are
> > completed/left)
> >
> >
> >  That would be good, though it'd probably be good to leave it as a
> > percentage instead of a "number of tests remaining".
> >
>
>
> AFAIK, any UI animation can only be done outside Flash (for example in JS),
> because Flash is single threaded.
>
> >
> > 6. Make some changes in text result presentation (for example errors will
> > be in red color, date will be bolded etc)
> >
> >
> >  Seems reasonable.
> >
>
> SGTM
>
> >
> > These of course are only proposals so feel free to comment
> >
> > Kind regards,
> > Sebastian Kostuch
> >
> >
> Thanks!
>
> >
> >
> >  Cheers,
> > Aaron
> >
> > Come visit the new www.internet2.edu
> >
> >