feat(huri): kube local example with TTS + EMAGE + RAG#33
Conversation
Release v0.1.0
bernTP
changed the title
…nce that's the job of Helm
…lm + custom health probe
…etching on client
…s swap capability
…cket and the dashboard
… avoid HF to redownload everytime models)
There was a problem hiding this comment.
Pourquoi pas créer plusieurs fichier ou alors réorganiser pour avoir une partie seulement class et une partie seulement function
| # pass pays the costs, the second confirms the path is hot), and | ||
| # synchronize so the GPU work is finished before we report ready. | ||
| # Best-effort: a failure here must never prevent the deployment coming up. | ||
| import time |
There was a problem hiding this comment.
import plutot au début du fichier
| gpu_mem_fraction: float = _GPU_MEM_FRACTION, | ||
| ): | ||
| print(f"[Gesture] importing torch...", flush=True) | ||
| import torch |
There was a problem hiding this comment.
import plutot au début du fichier
|
|
||
| def infer(self, audio_np: np.ndarray, source_sr: int = _EMAGE_SR) -> Motion: | ||
| import torch | ||
| import torch.nn.functional as F |
There was a problem hiding this comment.
import plutot au début du fichier
| import torch.nn.functional as F | ||
|
|
||
| if source_sr != _EMAGE_SR: | ||
| import librosa |
There was a problem hiding this comment.
import plutot au début du fichier
There was a problem hiding this comment.
pareille, import plutot au début du fichier
There was a problem hiding this comment.
import plutot au début du fichier
| # The following imports may contain modules with custom dependencies, depending on the Dockerfile | ||
| # CPU doesn't need some dependencies, nor the AMD that isn't compatible | ||
| try: | ||
| from src.modules.text_to_speech.text_to_speech import TTS |
There was a problem hiding this comment.
Je comprend pourquoi changer de fassons de faire mais pourquoi soit ne pas prendre l'ancienne, soit tout changer dans ce format ?
There was a problem hiding this comment.
(je supprime ce commantaire plus tard) mais regarde mes messages STP
MatthiasvonRakowski
left a comment
MatthiasvonRakowski
left a comment
There was a problem hiding this comment.
J'ai laissé quelques commentaires surtout sur les déclaration d'importe dans les function à titre personnel je ne trouve pas ça bien cat tu ne sais pas en entrant dans le fichier de quoi tu vas avoir besoin venant de endehors de ce fichier. Dans certain cas ça se justify, mais je préfèrerai mettre les imports, même celle qui sont en if - else, au début des fichier. Le reste me parrait bon, bravo super boulot.
3 participants
No description provided.