Finity-Alpha / OpenVoiceChat

Have a natural voice conversation with an LLM
http://www.finityalpha.com/OpenVoiceChat/
Apache License 2.0
39 stars 12 forks source link

Bounty - Pip installations #18

Closed SalehAhmad1 closed 2 months ago

SalehAhmad1 commented 2 months ago

I have setup pip installations for all packages.

Things to note:

  1. Packages being shared/used by one or more modalities were set as required packages like:

    • sounddevice
    • librosa
    • torch
    • transformers
    • torchaudio etc
  2. Packages being used by the TTS/LLM/TTS have been setup as individual installations. i.e. OpenVoiceChat[piper], OpenVoiceChat[xtts], OpenVoiceChat[llama] etc etc

I have also updated the Readme.md for better understanding of the user.

fakhirali commented 2 months ago

pip installing individual libs works.

I cannot do

import OpenVoiceChat as ovc

but the following works:

import llm.llm_gpt as llm_gpt

Maybe putting all of the things into an OpenVoiceChat module works? Idk. Ideally it is like the following:

import OpenVoiceChat as ovc
from ovc.llm import llm_gpt
SalehAhmad1 commented 2 months ago

Ok, I will update it then we'll test again.

SalehAhmad1 commented 2 months ago

Damnn, I didn't know it updates in the same PR without me sending one again xD Sorry for the mess!

fakhirali commented 2 months ago

Ez scene bro. Just commit to this branch and PR updates.

SalehAhmad1 commented 2 months ago

OVC can now be installed by using

Important Points:

fakhirali commented 2 months ago

Could you add transformers as an additional dependency as well. And also do the imports in init with llm_hf. I also feel torch is a heavy lib. When do we need it, can we remove it since transformers already requires it? Also we are probably version 0.1 lol

Bounty Completed! :moneybag:

SalehAhmad1 commented 2 months ago

Could you add transformers as an additional dependency as well. And also do the imports in init with llm_hf. I also feel torch is a heavy lib. When do we need it, can we remove it since transformers already requires it? Also we are probably version 0.1 lol

Bounty Completed! 💰

Files like main.py, translation.py etc import torch and use to setup compute device.

I'll add transformers as an optional dependency too, and edit the version too.

fakhirali commented 2 months ago

Hmm then leave torch in for now.