ndt-dev - Re: [ndt-dev] patches for the HTML5-frontend ?
Subject: NDT-DEV email list created
List archive
- From: Collin Anderson <>
- To: Athanasios Douitsis <>
- Cc: Aaron Brown <>, , Angela Dimitriou <>
- Subject: Re: [ndt-dev] patches for the HTML5-frontend ?
- Date: Fri, 4 Dec 2015 23:31:16 -0500
Aaron,
To hijack this thread a bit, hopeful to the benefit to Athanasios and others, would it be possible to maintain NDTjs as a separate repository in the NDT Project's Github account, and then link it with the main line through a submodule? I suspect that NDTjs will grow as we start to provide proper unit test coverage, and pay a bit more attention to the client in other ways. This might make soliciting contributions a bit more graceful, but more importantly, it would provide the opportunity to expose NDTjs in _javascript_ package managers like npm and run _javascript_-specific continuous integration on the repository.
Cordially,
Collin
On Mon, Oct 26, 2015 at 12:08 PM, Athanasios Douitsis <> wrote:
Re: firefox+websockets+workers: Sweet indeed, provided that it's accurate, which I don't know yet. But it works and the measurements . I am not using the NDTWrapper in ndt-wrapper.js but I rather have created a promise which invokes the worker in ndt-wrapper-ww.js, regardless of browser type. Works and produces similar results in firefox and chrome, haven't tested elsewhere. Do you have a formal/semiformal way of ascertaining that a measurement is accurate enough?Hello Aaron, thanks for the reply!,Re: websockets: I think that I might be eventually able to come up with a small patch, but it may be slightly more complex that the one liner I envisaged. No matter though, yes, I'll issue a pull request if and when I do. My tests already play and detect failures seemingly correctly, but I'll try to think about possible corner cases before rushing it out the door.Kind regards,Athanasios--On Thu, Oct 22, 2015 at 7:57 PM, Aaron Brown <> wrote:
Hi,
On Oct 21, 2015, at 11:06 AM, Athanasios Douitsis <> wrote:Hello,In case one desires to submit a small patch (e.g. for the ndt-browser-client.js) under HTML5-frontend, would a pull request on https://github.com/ndt-project/ndt be appropriate?You stated it wouldn't be easy but if you do come up with one, that's the easiest way.I have run across an mlab ndt server that for some reason does not respond normally to the websocket version of the protocol. While this sounds bad, it actually gave me the opportunity to realize that, upon creation of the websocket in ndt-browser-client.js, the onclose and onerror events are not handled, so if the connection is dropped abnormally or any other sort of error occurs, the application using NDTjs doesn't learn it and can hang in there indefinitely.Also, another question, in my tests I am using Workers under Firefox quite successfully. It seems that the bug saying that WebSockets were not available in Workers (*) is nowadays fixed. Anyone aware whether there is any issue concerning the accuracy of measurements? Because as far as I can tell it works beautifully.
If I may be permitted, I'd like to submit a small patch to fix that.Sweet. I guess it's a more recent version. If you'd be interested in patching that issue, we'd be happy to accept it. Otherwise, I'm happy to take a look at fixing it.Cheers,AaronAthanasios Douitsis
- Re: [ndt-dev] patches for the HTML5-frontend ?, Collin Anderson, 12/05/2015
- Re: [ndt-dev] patches for the HTML5-frontend ?, Aaron Brown, 12/07/2015
Archive powered by MHonArc 2.6.16.