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

[WIP] Create a binary decoder with a TART reasoning module #3730

Draft
wants to merge 15 commits into
base: tart-development
Choose a base branch
from

Conversation

jeffkinnison
Copy link
Contributor

@jeffkinnison jeffkinnison commented Oct 13, 2023

This adds a TART reasoning module as a binary decoder.

TODOs:

[] Add the vanilla and LOO embedding protocols
[] Add an embeddings cache
[] Tests

@jeffkinnison jeffkinnison changed the base branch from master to tart-development October 13, 2023 22:08
@github-actions
Copy link

Unit Test Results

  6 files  ±0    6 suites  ±0   20m 46s ⏱️ -22s
12 tests ±0    9 ✔️ ±0    3 💤 ±0  0 ±0 
60 runs  ±0  42 ✔️ ±0  18 💤 ±0  0 ±0 

Results for commit 2c28ac8. ± Comparison against base commit b6bdf9c.

@ANarayan
Copy link
Collaborator

@jeffkinnison can we pass the tokenizer in to the embedding protocols?

@jeffkinnison
Copy link
Contributor Author

@ANarayan Let's assume that we can pass in the tokenizer and model into the protocol functions. That should simplify initial development, and we can revisit that design if needed. Also, feel free to update the function signatures or convert them into classes, or to make any other changes!

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.

None yet

2 participants