Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Urgent, change temp file folder from "tmp" to mkdtemp #14

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Urgent, change temp file folder from "tmp" to mkdtemp #14

wants to merge 3 commits into from

Commits on Apr 13, 2023

  1. Update inference.py 

    changes required to use tempfile mkdtemp()

original code would potentially wipe out tmp -named directory on destination video's folder.
    @anttiryt
    anttiryt committed Apr 13, 2023
    Configuration menu
    Copy the full SHA
    7b8dd8b View commit details
    Browse the repository at this point in the history

  2. inference, renamed var output_filename to data_path 

    Renamed output_filename to data_path in extract_frames() for consistency with other code.
    @anttiryt
    anttiryt committed Apr 13, 2023
    Configuration menu
    Copy the full SHA
    ea1a114 View commit details
    Browse the repository at this point in the history

  3. inference extract_frames, removed extra parameter output_path 

    removed output_path, is not used
    @anttiryt
    anttiryt committed Apr 13, 2023
    Configuration menu
    Copy the full SHA
    1394f1a View commit details
    Browse the repository at this point in the history