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

Dependencies: Switch to pyproject.toml #88

Merged
merged 9 commits into from
Mar 20, 2024
Merged

Dependencies: Switch to pyproject.toml #88

merged 9 commits into from
Mar 20, 2024

Conversation

bdashore3
Copy link
Member

This PR switches from requirements.txt to pyproject.toml for dependencies and configuration. The main reason is because maintaining multiple requirements files is difficult and time consuming. In addition, update the start scripts to reflect these changes and prompt the user for their GPUs.

This will manage dependencies from now on since it's a more flexible
file that's similar to other packaging utilities like npm and cargo.

Signed-off-by: kingbri <[email protected]>
Pyproject.toml replaces these files.

Signed-off-by: kingbri <[email protected]>
Removes unnecessary ruff.toml.

Signed-off-by: kingbri <[email protected]>
v0.0.16

Signed-off-by: kingbri <[email protected]>
When the model is processing a prompt, add the ability to abort
on request cancellation. This is also a catch for a SIGINT.

Signed-off-by: kingbri <[email protected]>
Some tokenizer variables don't get cleaned up on init, so these can
persist. Clean these up manually before creating a new tokenizer for
now.

Signed-off-by: kingbri <[email protected]>
There is no platform agnostic way to fetch CUDA/ROCm's versions
since environment variables change and users don't necessarily need
CUDA or ROCm installed to run pytorch (pytorch installs the necessary
libs if they don't exist).

Therefore, prompt the user for their GPU lib and store the result in
a textfile so the user doesn't need to constantly enter a preference.

Signed-off-by: kingbri <[email protected]>
Adopt to new pyproject install method

Signed-off-by: kingbri <[email protected]>
@bdashore3 bdashore3 merged commit 51b289c into main Mar 20, 2024
1 check 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.

1 participant