Remove update_ago_timer variable and update_ago function.

[dongjoon-hyun]

Start Time shows the start time of the job and doesn't seem to use these any more.
The timer calls this function every second (in comments it should be 5),
but the function update_ago just returns without doing anything.

[lukeyeager]

@jmancewicz - any reason to keep these around?

[jmancewicz]

It updates the duration for running jobs on the front page.

[lukeyeager]

What about this return statement though:
https://github.com/NVIDIA/DIGITS/blob/v3.1.0/digits/templates/job_management.html#L41

[jmancewicz]

Ah. I don't recall. I'll have to look into this today. I do recall that there was some issue with getting the time to feel right.

[dongjoon-hyun]

Hi! :-)

[dongjoon-hyun]

Ping! :)

[dongjoon-hyun]

Hi, @lukeyeager and @jmancewicz .
It's okay to close this PR if the code is intentionally designed like that.
Just leave a message for a record. :)

[dongjoon-hyun]

Rebased.

[lukeyeager]

I just looked into this and it looks like Joe disabled the "ago" thing with #380 which was originally added with #240.

I'm in favor of just deleting all of the "ago"-related stuff, including two files in static/js/:

lukeyeager@f3746f4

@jmancewicz does that look right to you? Do you think we need to figure out how to fix the time-sync issues or can we just abandon the feature?

[dongjoon-hyun]

Oh, that sounds great to me.
I can handle all of them by updating this PR if you want.

[jmancewicz]

That looks great. Good to have it cleaned out.

[lukeyeager]

Thanks for getting back to us @jmancewicz. @dongjoon-hyun I'll let you update this PR unless you'd rather me just do it myself.

[dongjoon-hyun]

Thank you, @lukeyeager . It's my pleasure. I'll update this PR soon. :)

[dongjoon-hyun]

Oh, @lukeyeager .
After looking back the code and yours, I realized that you've already finished two days ago what you want in the Remove cruft for printing 'X time ago' on homepage.

If that is true, you can close my PR and merge yours. It's no problem. I respect DIGITS upstream. :)
+1 for your commit. :)

[lukeyeager]

Closed in favor of #602.