Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Cleaning up#42

Merged
shabani1 merged 3 commits intomainfrom
rs/minor-cleanup
Jan 9, 2024
Merged

Cleaning up#42
shabani1 merged 3 commits intomainfrom
rs/minor-cleanup

Conversation

@shabani1
Copy link
Contributor

@shabani1 shabani1 commented Jan 9, 2024

What

IMPORTANT: run alembic upgrade head after pulling

  • Removed the unused Embedding model and the embedding endpoints
  • Cleaned up the transformer endpoints
    • Got rid of anything that imported a task directly
    • Can use transform_document endpoint to call any transformer instead
  • Minor edits to documentation

Why

The people don't like unused code sitting around.

Test plan

  • Run pytest sdk-python
  • Run examples/tests.ipynb and verify that there are no errors
  • Run examples/tutorial.ipynb and verify that the tutorial works as expected
  • Run examples/images.ipynb and verify that the tutorial works as expected

@shabani1 shabani1 merged commit a1815d1 into main Jan 9, 2024
@shabani1 shabani1 deleted the rs/minor-cleanup branch January 9, 2024 23:37
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

Comments