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

Add document-question-answering in task_summary #23252

Closed
wants to merge 18 commits into from

Conversation

y3sar
Copy link
Contributor

@y3sar y3sar commented May 10, 2023

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

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented May 10, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Member

@stevhliu stevhliu left a 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).

y3sar and others added 5 commits May 11, 2023 09:59
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
@y3sar y3sar force-pushed the add-docqa-example-to-task_summary branch from c735c4e to 396e031 Compare May 11, 2023 04:05
@y3sar
Copy link
Contributor Author

y3sar commented May 11, 2023

@stevhliu I only changed the docs. I don't know why the checks keep failing. It says it is having problems with ffmpeg?

younesbelkada and others added 11 commits May 11, 2023 09:32
* 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>
Copy link
Member

@stevhliu stevhliu left a 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?

@y3sar
Copy link
Contributor Author

y3sar commented May 11, 2023

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>
@y3sar
Copy link
Contributor Author

y3sar commented May 12, 2023

I created a new pull request #23318 closing this PR

@y3sar y3sar closed this May 12, 2023
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.

6 participants