open-mmlab / Amphion

Amphion (/Γ¦mˈfaΙͺΙ™n/) is a toolkit for Audio, Music, and Speech Generation. Its purpose is to support reproducible research and help junior researchers and engineers get started in the field of audio, music, and speech generation research and development.
https://openhlt.github.io/amphion/
MIT License
5.91k stars 452 forks source link

Windows compatibility #136

Open HighPriest opened 8 months ago

HighPriest commented 8 months ago

✨ Description

As this is a Python application, there shouldn't be a reason for it to not be OS agnostic. Hence this PR.

🚧 Related Issues

πŸ‘¨β€πŸ’» Changes Proposed

πŸ§‘β€πŸ€β€πŸ§‘ Who Can Review?

πŸ›  TODO

βœ… Checklist

HarryHe11 commented 8 months ago

Hi @HighPriest, Thank you very much for submitting this valuable PR! Please don't hesitate to engage in further discussions with @RMSnow, @lmxue, and @VocodexElysium regarding any potential changes to this PR.

RMSnow commented 8 months ago

Hi @HighPriest, thanks for your efforts! I reviewed your code. The design including replacing .sh to .py and refine the os path operation is worth of following by Amphion.

Amphion team has been actively exploring the best ways to support our Windows users. One viable approach is certainly the one you've proposed, which allows users to utilize their local windows machines to train a model from scratch. We are also considering alternative solutions such as that we only support basic education features such as visualization tools, Jupyter Notebook, interactive demo, and pretrained models for users. For those who want to utilize their local GPU, we provide only the tutorial of Linux-based virtual machine installation, and sill recommend them to use Linux platform to train neural networks. This is a challenging issue that touches on the core philosophy of our product, and we are yet to reach a definitive decision.

We truly value your suggestions and efforts to enhance the functionality of Amphion. As an interim solution, we will recommend Windows users to leverage your branch in this PR as a patch. Thank you so much for your contribution!

HighPriest commented 8 months ago

@RMSnow Please do not encourage anyone to use this PR as a functional solution for windows.

This PR has been marked as a draft! It doesn't complete the first step (features extraction), because of missing dependency on Windows.

RMSnow commented 8 months ago

@HighPriest Oh, Sure! Thank you all the same :)

HppZ commented 3 months ago

πŸ˜’