Custom Insights Example incorrect. about azure-video-indexer-samples

[nofaredanoren]

Hey @Nico-VanHaaster
We found the bug and working on a fix. Will update you once resolved.

Thanks for letting us know!

[Nico-VanHaaster]

@nofaredanoren Thanks for the update. Do you have any estimated timeframe for the fix?

[nofaredanoren]

You're welcome @Nico-VanHaaster
The fix was deployed, can you please verify from your side?

Thanks

[Nico-VanHaaster]

@nofaredanoren Thanks again for the update and although the NaN:NaN:NaN is resolved I dont believe its actually correct. Looking at the new minified code it is handling the undefined correctly however now the video duration is now returning (from the example of duration: 634 duration in seconds) the value "634:00:00" which would indicate the video is 634 hours not 10 mins 34 seconds.

Code like below should solve this.

const duration = 634;
const hour = Math.floor(duration / (3600));
const minutes = Math.floor(duration / 60) % 60;
const seconds = duration % 60;

const durString = [hour, minutes, seconds]
  .map(t => String(!isNaN(t) ? t : 0).padStart(2, "0"))
  .join(":");

console.log(durString);

[nofaredanoren]

Thanks @Nico-VanHaaster for the suggestion! The widgets works a bit different and I appreciate your help.
Another fix was deployed, can you check again?
Thanks

[Nico-VanHaaster]

@nofaredanoren Great thank you very much. I just tested the sample and our own usage of custom insights and this seems to be resolved. I really appreciate your quick turn around on these fixes.

I will close this for bug report.