Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • bob.bio.video bob.bio.video
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • bobbob
  • bob.bio.videobob.bio.video
  • Merge requests
  • !34

Returning none insted of empty FrameContainer in preprocessor Wrapper

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Olegs NIKISINS requested to merge preproc_wrapper into master Mar 13, 2018
  • Overview 2
  • Commits 1
  • Pipelines 1
  • Changes 2

This update returns None instead of empty FrameContainer in the preprocessor Wrapper class for an extreme case (for example, wrapped preprocessor returned None for all frames in video for some reasons).

Works well with --allow-missing-files option, because preprocessed file will not be written in this case.

Currently, empty FrameContainer is written to file, which might cause an error in the Extractor, if not handled. Also, added the test for this case.

@mguenther or @amohammadi, please review and merge if OK with you. Thank you!

@ageorge, can be useful for you as-well.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: preproc_wrapper