NDT-DEV email list created

[Sebastian Kostuch <>]

Hi Tiziana,
Thank you so much for all your reviews.
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?
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.
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 hope my thoughts are clear. If not, feel free to ask any questions :)

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
> >
> >