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

feat(25): make models optionals #27

Merged
merged 2 commits into from
Apr 26, 2024
Merged

feat(25): make models optionals #27

merged 2 commits into from
Apr 26, 2024

Conversation

TheJoin95
Copy link
Member

Closes #25

This PR will make the .pt files optionals and loadable from this repository.
On the latest release the package was having a 30MB size standalone and more than 700 MB if we consider the dependencies.

We should let the user decide which algorithm use for converting an image.
After this PR, we're going to release a minor.

@TheJoin95 TheJoin95 requested a review from Wabri April 24, 2024 22:13
@TheJoin95 TheJoin95 merged commit ed78105 into master Apr 26, 2024
@TheJoin95 TheJoin95 deleted the feat/25 branch April 26, 2024 08:33
@Wabri
Copy link
Member

Wabri commented Apr 27, 2024

you have no patience 😆

@TheJoin95
Copy link
Member Author

@Wabri I'm sorry broski ma già a lavoro ho I bottle neck di ownership che mi fan girare inutilmente le balls e sclerando ho mergiato.

Could you please add the protection on the branch?

@Wabri
Copy link
Member

Wabri commented Apr 27, 2024

No problem brothersky, I saw the code and I didn't find any major problem. Thanks to have time to develop 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

Successfully merging this pull request may close these issues.

Add a parameter to load a model for the conversion method
2 participants