Don't use external commands #77

New Issue

nekobit · 2022-12-17T01:32:13Z

nekobit commented

2022-12-17 01:32:13 +00:00

At the moment, rDrama is heavy on external command usage, specifically ImageMagick. I can't account for the performance downsides of this, but using the real API is much prefered, could reduce bugs too.

It would be better if we accessed direct API's as Python bindings for ImageMagick (i.e., other commands are being externally called) purely exist. For fucks sake, we're installing X11 in our docker package and wayland because Debian likes grabbing as much shit as possible.

I would imagine for some commands we're parsing the stdout (output) contents, let's just... mmm, use the api's directly which can get the right info anyway.

At the moment, rDrama is heavy on external command usage, specifically ImageMagick. I can't account for the performance downsides of this, but using the real API is much prefered, could reduce bugs too. It would be better if we accessed direct API's as Python bindings for ImageMagick (i.e., other commands are being externally called) purely exist. For fucks sake, we're installing X11 in our docker package and wayland because Debian likes grabbing as much shit as possible. I would imagine for some commands we're parsing the `stdout` (output) contents, let's just... mmm, use the api's directly which can get the right info anyway.

👍 1

mummified-corroding-granny commented

2023-02-15 05:46:03 +00:00

I have looked at this briefly at this:

ffmpmeg subprocess calls: there is a popular ffmpeg library and pip package but the main developer might have gone looney tunes as per twitter. The package has not been updated in a year and has many hundreds of PRs + issues
Debian being Debian
ImageMagick has been solved, I think

I have looked at this briefly at this: * ffmpmeg subprocess calls: there is a popular ffmpeg library and pip package but the main developer might have gone looney tunes as per twitter. The package has not been updated in a year and has many hundreds of PRs + issues * Debian being Debian * ImageMagick has been solved, I think

Aevann closed this issue

2023-08-12 10:54:45 +00:00

Sign in to join this conversation.