Skip to content

Rationale #48

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Rationale #48

wants to merge 4 commits into from

Conversation

schopra6
Copy link
Collaborator

Still in improvement phase
readme file has instructions to run the flan t5 fine-tuned model

@schopra6 schopra6 requested a review from nfelnlp May 15, 2023 21:57
@nfelnlp nfelnlp added the enhancement New feature or request label May 16, 2023
@nfelnlp
Copy link
Member

nfelnlp commented May 17, 2023 •
edited
Loading

Looks great! Could you upload the fine-tuned model to the DFKI cloud?
For the next steps, it's more important to use your existing one instead of training a new one. Then we can extend the rationalize operation by adding an option to use the FLAN-T5 instead of the GPT-Neo.

@nfelnlp nfelnlp removed this from the Preparation for human evaluation milestone May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nfelnlp nfelnlp Awaiting requested review from nfelnlp

Assignees

@schopra6 schopra6

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@schopra6 @nfelnlp