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

Fix mecab-python3 version #159

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

polm
Copy link

@polm polm commented Jun 30, 2024

Hello, I am the maintainer of mecab-python3. Users of your library have been opening issues at my repo because they have trouble building MeCab. They should not have to build MeCab, but MeloTTS is using an old version of mecab-python3 that doesn't support recent Python versions. This raises the version. There are no functionality changes.

mecab-python3 1.0.5 is from early 2022 and doesn't have 3.11 wheels.
1.0.9 is more recent and has wheels for all platforms.

Also mecab-python3 was specified twice for some reason.
@polm
Copy link
Author

polm commented Jun 30, 2024

This also removes the double entry noted in #150.

@prateeksachan
Copy link

prateeksachan commented Jul 3, 2024

One of our repo is also receiving issues related to this. thanks @polm for providing the patch. ref: https://github.com/bolna-ai/bolna/issues/305

@polm
Copy link
Author

polm commented Jul 4, 2024

Also see #51 - mecab-python3 should be optional in this library, since not everyone is using Japanese.

unok pushed a commit to unok/MeloTTS that referenced this pull request Jul 10, 2024
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 23.12.1 → 24.4.2](psf/black@23.12.1...24.4.2)
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

3 participants