Feature/voice analysis #30
Conversation
… system and heavy file constraints
DnzzL
force-pushed
the
feature/voice_analysis
branch
from
6177b88
to
f9ffc8e
Compare
DnzzL
force-pushed
the
feature/voice_analysis
branch
from
dd79922
to
ecf17c6
Compare
DnzzL
force-pushed
the
feature/voice_analysis
branch
from
ecf17c6
to
fe3f487
Compare
There was a problem hiding this comment.
Hello,
Pas mal de choses à revoir dans cette PR @Lokhia @DnzzL avant de la merge sur main. Le plus important c'est qu'elle ne respecte pas pour le moment les conventions du repo c'est à dire en philosophie librairie avec des modules interopérables. Il faut qu'on puisse facilement pouvoir faire
``from bechdelai.audio import ...```
Sinon ça va être impossible à déployer et à rendre interopérable avec le reste.
Les autres commentaires sont dans les fichiers.
Aussi on va rajouter une règle, il faut qu'on ait un notebook qui explique les fonctionnalités dans le dossier racine tutorials avant d'être validé sur main (parce qu'à terme c'est ce qui partira en prod sur PyPi).
En attendant sur la partie audio pour la démo restons sur la branche feature/voice_analysis le temps de le faire évoluer proprement :)
pyproject.toml
Outdated
| @@ -1,12 +1,12 @@ | |||
| [tool.poetry] | |||
| name = "bechdelai" | |||
| version = "0.0.1-alpha.2" | |||
| version = "0.1.0" | |||
There was a problem hiding this comment.
Restons sur une version 0.0.2 pour le prochain bump
| openpyxl = "^3.0.10" | ||
| pytube = "^12.1.0" | ||
| mediapipe = "^0.9.0" | ||
| scenedetect = {extras = ["opencv"], version = "^0.6.1"} |
There was a problem hiding this comment.
Il faut garder ces versions dans le merge
| columns=['gender', 'start', 'end']) | ||
|
|
||
|
|
||
| def _convert_whisper_output(self,segments:pd.DataFrame) -> list: |
There was a problem hiding this comment.
je trouve le nom de la fonction pas hyper clair
| print("Could not request results from Google Speech Recognition service; {0}".format(e)) | ||
|
|
||
|
|
||
| class WhisperAPI(Transcriber): |
There was a problem hiding this comment.
j'aimerais bien de la docstring pour clarifier l'usage entre WhisperAPI et Whisper
| from moviepy.video.io.ffmpeg_tools import ffmpeg_extract_subclip | ||
|
|
||
|
|
||
| def cut_and_save(movie_path: str, start: float, end: float, target_name: str) -> None: |
notebooks/audio/OLD/.env.example
Outdated
| path_to_extract=<path_to_extract_file> | ||
| path_to_audio=<path_to_audio_file> | ||
| path_to_full_movie=<path_to_full_movie_file> | ||
| path_to_trailer=<path_to_trailer_file> |
notebooks/audio/OLD/README.md
Outdated
| #Readme pour l'audio | ||
| ## Installation | ||
| * .env | ||
| * créer un fichier ".env" en local pour y placer le chemin vers la vidéo, comme dans .env.example | ||
| * poetry update / poetry install | ||
| * poetry run python .\gender_identification.py | ||
| * ffmpeg codex (pour Windows, suivre les instructions [ici](https://www.geeksforgeeks.org/how-to-install-ffmpeg-on-windows/) - | ||
| pas besoin de mettre à la racine en admin et de redémarrer ; | ||
| pour ubuntu `$ sudo apt-get install ffmpeg`, voir la [doc du projet](https://github.com/ina-foss/inaSpeechSegmenter)) |
No description provided.