Add TrackJSON Download and SkipTranscoding Options. by BryonLewis · Pull Request #1291 · Kitware/dive

@BryonLewis

  • Annotation download button modified within web version to have a dropdown to select the TrackJSON instead of always exporting the viame-csv format
  • Logic included preventing transcoding when uploaded with the skipTranscoding parameter set to true. Should only be done by those who know that their video is h264 encoded.

@waxlamp

  • Logic included preventing transcoding when uploaded with the skipTranscoding parameter set to true. Should only be done by those who know that their video is h264 encoded.

What happens if you knowingly enable this option on a video that's not in the right format? (i.e., what kind of error message or error state will the user find themselves in?)

@BryonLewis

  • Logic included preventing transcoding when uploaded with the skipTranscoding parameter set to true. Should only be done by those who know that their video is h264 encoded.

What happens if you knowingly enable this option on a video that's not in the right format? (i.e., what kind of error message or error state will the user find themselves in?)

Currently, it will fall back on transcoding the video. That is probably not desired when importing videos and having that option set. I can easily change it to create an error in the job log.

@BryonLewis

Note to self: Clarify the setting as being "Attempt to Skip transcoding". It will fall back to transcoding if the video files are not in the proper format. Log this information in the Job log and update the UI elements to accurately indicate to the user this setting (may need to use a help icon tip for the longer explanation).

@BryonLewis

marySalvi

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, working well. Just minor grammar changes.

Co-authored-by: Mary Salvi <40494088+marySalvi@users.noreply.github.com>
Co-authored-by: Mary Salvi <40494088+marySalvi@users.noreply.github.com>

marySalvi

@BryonLewis

marySalvi