-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
Add document-question-answering in task_summary #23252
Conversation
The documentation is not available anymore as the PR was closed or merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, thanks for taking this up! 🤗
This is more of a multimodal task, so I think it'd be better to create a new ## Multimodal
section for this task (kind of like this).
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
c735c4e
to
396e031
Compare
@stevhliu I only changed the docs. I don't know why the checks keep failing. It says it is having problems with ffmpeg? |
* fix gpt2 inference * fixup * no need to be in `_keys_to_ignore_on_load_missing`
…ace#23257) * Temp tol fix for flaky whipser test * Add equivalent update to TF tests
…uggingface#22787) * update min k_value of conditional detr post-processing * feat: add top_k arg to post processing of deformable and conditional detr * refactor: revert changes to deprecated methods * refactor: move prob reshape to improve code clarity and reduce repetition
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a suggestion to simplify the multimodal section a bit and to break up the text so it is more readable. Otherwise, looks good to me!
Looks like your merge
may have introduced some other changes. I think maybe closing this and opening a new PR would the easiest here? What do you recommend @amyeroberts?
Thank you so much for you help @stevhliu I think I should open a new PR. I made a mess here |
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
I created a new pull request #23318 closing this PR |
What does this PR do?
From issue #18926
This PR adds Document Question Answering summary to task_summary.mdx
It also provides an example using pipeline and this model
Who can review?
@stevhliu