Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

requirement to split out machatts-core #130

Open
JohnHerry opened this issue Dec 26, 2024 · 1 comment
Open

requirement to split out machatts-core #130

JohnHerry opened this issue Dec 26, 2024 · 1 comment

Comments

@JohnHerry
Copy link

Firstly thanks to the good job of matcha,
But in my view of point, there are too many dependence to install the wheel package.
there are many Unnecessary third-party packages in requirements.txt, like gradio, torchvision, or other vision tools. that may help view the training process and inference test, but also lead to complicated package dependence and make it hard to install with simple pip in a pre-installed env.
My suggestion is a pure matcha-core installation requirement list, so that the core algorithm part can be installed easily, and can be used generally.
say sorry if the suggestion is not thoughtful to this project.

@shivammehta25
Copy link
Owner

That is a good suggestion, I will implement it. However, due to other commitments right now. I am having difficulty actively maintaining the package, but I accept PRs. Would love it if you or something could help with it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants