in03 / proxima

Transcode source media directly from DaVinci Resolve using multiple machines for encoding. Great for creating proxies quickly.
MIT License
53 stars 3 forks source link

This was causing the worker start cmd to fail after changing to absolute imports #223

Open github-actions[bot] opened 2 years ago

github-actions[bot] commented 2 years ago

This was causing the worker start cmd to fail after changing to absolute imports

Not sure why we needed it in the first place? Would be good to do further testing and see

if it is necessary in some cases.

cwd=get_module_path(),

https://github.com/in03/proxima/blob/9dc90d2100defd3f8cfb1912e6204bc43e9bff25/proxima/worker/launch_workers.py#L169


    logger.debug(f"[magenta]{' '.join(launch_cmd)}[/]\n")

    subprocess.Popen(
        # TODO: This was causing the worker start cmd to fail after changing to absolute imports
        # Not sure why we needed it in the first place? Would be good to do further testing and see
        # if it is necessary in some cases.
        # labels: testing
        # cwd=get_module_path(),
        args=" ".join(launch_cmd),
        shell=True,
    )

575da7478368317a24b6c52c113269b265724651