Discussion :

[fred1599]

Bonjour,

Suite au code proposé par Tyrtamos : https://www.developpez.net/forums/d2...-lance-python/

J'avais commencé dans la réponse à son application une proposition pour contribuer à son projet en le rendant plus conventionnel. Étant donné que je n'aime pas perdre trop de temps pour rien, j'ai souhaité garder mon code sur le net, c'est pourquoi je propose de le rendre visible sur github et permettra à tout à chacun de proposer des améliorations pour avancer sur le sujet et le rendre utilisable par tous...

Tout est indiqué sur la page principale du projet GitHub afin de participer et proposer vos idées (anglais et français).
Pour l'utilisation c'est en anglais, mais c'est très simple à comprendre (python3.10 -m video --help) et vous aurez tous les éléments.

Je trouve cette idée sympa pour les expérimentés ou pas d'ailleurs de partager leurs compétences techniques.

Lien du projet : https://github.com/fred1599/video_project

Prochaines étapes :

1. Amélioration des tests unitaires (fait rapidement, voir trop rapidement)
2. Tests de l'application sur l'OS Windows (j'ai pas Windows) et compléter GitHub Actions pour Windows (je crois qu'il faut utiliser choco pour installer ffmpeg)
3. Ajout de pre-commit pour l'uniformisation du développement
4. Ajout de mypy et des typing
5. Autres (à vous de proposer des évolutions)
6. Développement de la classe VideoList

À bientôt peut-être pour vos futures contributions

[papajoker]

bonjour

J'avais noté 2 choses dans ton code:

is_tool_installed()
Nous avons en python shutil.which("app_avec_ou_sans_path").

-----------

result = subprocess.run()
return output_file

Attente de la sortie Avec le traitement d'une vidéo, on va attendre des minutes sans rien à l'écran ...

par exemple, avoir une méthode générateur de ce type

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
class Video:
    @staticmethod
    def _run(command: list) -> Generator[str, Any, None]:
        process = subprocess.Popen(
            command,
            stdout=subprocess.PIPE,
            stderr=subprocess.PIPE,
            #shell=True,
            text=True,
        )
        while True:
            line = process.stdout.readline().rstrip()
            if line == '' and not process.poll():
                break
            yield line
        if err := process.stderr.read():
            raise Exception(f"Erreur {command[0]}: {err}")
 
    def convert(self, output_format: str, quality: str = "middle") -> Optional[str]:
        command = [shutil.which("ffmpeg"), "-i", self.filepath, "-crf", crf, output_file]
        yield from self._run(command):

afin de participer

Je n'ai jamais fait de conversion vidéo et n'en ai aucun besoin

[fred1599]

Hello @papajoker,

Ce qui est dommage c'est de ne pas contribuer sur GitHub et proposer une PR, mais je vais regarder ta proposition de code, merci !

Je n'ai jamais fait de conversion vidéo et n'en ai aucun besoin

Perso moi non plus, et professionnellement, tes applis tu ne les utilises pas personnellement et n'ont aucun intérêt sinon d'avoir un salaire.

Dès que j'ai du temps de dispo, je vérifie ta proposition...