Skip to content

Conversation

@bkb2135
Copy link
Collaborator

@bkb2135 bkb2135 commented Nov 26, 2024

Changes

  • Switch from Bittensor communication protocol to Epistula.
  • Switch from vLLM to transformers architecture.
  • Switch from async generators to OpenAI httpx clients for streaming.

Other

  • Fix the whole codebase style/format/isort issues to align with ruff, black, isort.
    • Add ruff, black, isort.
  • Add pre-commit hook, pre-commit checks on pull-request with Github Actions.
    • Add contribution guide to CONTRIBUTING.md.
    • Remove Llama 8B model.

Overview

Each git commit will run pre-commit hook locally, and fix all format/style issues.
If commit fails, make sure to add the modified files.

To run pre-commit manually and automatically fix files: poetry run pre-commit run --all-files


bkb2135 and others added 2 commits November 26, 2024 08:18
## Changes
  - Switch from Bittensor communication protocol to Epistula.
  - Switch from vLLM to transformers architecture.
  - Switch from async generators to OpenAI httpx clients for streaming.
  
## Other
- Fix the whole codebase style/format/isort issues to align with ruff,
black, isort.
  - Add ruff, black, isort.
- Add pre-commit hook, pre-commit checks on `pull-request` with Github
Actions.
  - Add contribution guide to `CONTRIBUTING.md`.
  - Remove Llama 8B model.

## Overview
Each `git commit` will run pre-commit hook locally, and fix all
format/style issues.
If commit fails, make sure to add the modified files.

To run pre-commit manually and automatically fix files:
`poetry run pre-commit run --all-files`

---------

Co-authored-by: richwardle <[email protected]>
Co-authored-by: richwardle <[email protected]>
Co-authored-by: Dmytro Bobrenko <[email protected]>
Copy link
Contributor

@dbobrenko dbobrenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bkb2135 bkb2135 merged commit 7497354 into main Nov 26, 2024
2 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.

3 participants