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

added moshi-* scripts/entrypoints to pyproject.toml #73

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

ErikBjare
Copy link
Contributor

@ErikBjare ErikBjare commented Sep 19, 2024

Checklist

  • Read CONTRIBUTING.md, and accept the CLA by including the provided snippet. We will not accept PR without this.
  • Run pre-commit hook.
  • If you changed Rust code, run cargo check, cargo clippy, cargo test.

CLA

I, Erik Bjäreholt (@ErikBjare), confirm that I have read and understood the terms of the CLA of Kyutai-labs, as outlined in the repository's CONTRIBUTING.md, and I agree to be bound by these terms.

PR Description

With this change you can now install moshi with:

  • pipx install moshi (once it's on PyPI)
  • pipx install -e ./moshi/moshi (which I tested)

This wasn't possible before since pipx requires a valid entrypoint to allow installation.

moshi_mlx/pyproject.toml Outdated Show resolved Hide resolved
@adefossez
Copy link
Collaborator

Also you need to agree to the CLA by following the instructions in contributing.md

@ErikBjare ErikBjare force-pushed the dev/add-pyproject-scripts branch from 646f664 to cdc80c3 Compare September 19, 2024 14:29
@ErikBjare
Copy link
Contributor Author

Alright, fixed it up and got it running now. Very cool!

Let me know if there's anything else I should do.

moshi_mlx/pyproject.toml Outdated Show resolved Hide resolved
@ErikBjare ErikBjare changed the title added moshi-server scripts/entrypoints to pyproject.toml added moshi-* scripts/entrypoints to pyproject.toml Sep 19, 2024
@ErikBjare ErikBjare force-pushed the dev/add-pyproject-scripts branch from cdc80c3 to 1d864b7 Compare September 19, 2024 14:37
moshi_mlx/pyproject.toml Outdated Show resolved Hide resolved
@adefossez
Copy link
Collaborator

final nit before I can accept

@ErikBjare ErikBjare force-pushed the dev/add-pyproject-scripts branch from 1d864b7 to f03dc61 Compare September 19, 2024 17:13
@ErikBjare
Copy link
Contributor Author

Fixed!

@adefossez adefossez merged commit ae34219 into kyutai-labs:main Sep 20, 2024
3 checks passed
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.

2 participants