Chore: Removed the use of the "delay" constant as it is no longer used when starting monitors on the Montage page

[IgorA100]

Probably it broke at some point. Or maybe it never worked, but in montage.js monitors start with a delay: "monitor.start(delay);"
Essentially, the changes consist only of wrapping the function code in

    setTimeout(function() {
      ......
    }.bind(this), delay);

and adding spaces

[connortechnology]

We don't want to delay loading the video, that makes for bad UI. The server has to be able to handle the load no matter what. The entire idea of delaying the streaming is flawed I think. Given the single-threaded nature of browsers, I think it doesn't have an effect anyways.

The delay was originally for the status update ajax streams, not the actual video. I switched from a self-refreshing setTimeout to an Interval. Beacuse we were getting into a state where we have multiple setTimeouts running. The Interval is far simpler.

[IgorA100]

that makes for bad UI

In principle I agree.

I switched from a self-refreshing setTimeout to an Interval.

Yes, I saw that.
So the code in montage.js should be changed from

      // Start the fps and status updates. give a random delay so that we don't assault the server
      const delay = Math.round( (Math.random()+0.5)*statusRefreshTimeout );
      monitors[i].start(delay);

on
monitors[i].start();

And in MonitorStream.js remove the "delay" argument from function. Otherwise there will be some confusion.

[connortechnology]

yes

[IgorA100]

Done