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

Squad inference example #390

Merged
merged 4 commits into from
Feb 20, 2020
Merged

Squad inference example #390

merged 4 commits into from
Feb 20, 2020

Conversation

yzhang123
Copy link
Contributor

No description provided.

Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
@yzhang123 yzhang123 requested a review from ekmb February 20, 2020 17:18
Comment on lines +63 to +64
"start_positions": NeuralType(tuple('B'), ChannelType(), optional=True),
"end_positions": NeuralType(tuple('B'), ChannelType(), optional=True),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@okuchaiev What exactly does optional mean? How does it work if I only return end_positions but not start_positions?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

start_positions, end_positions are the "labels", they are optional to make the data_layer more general for inference cases where i do not have the groundtruth.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense, I just didn't know we had this option, and want to know how it works.

Copy link
Collaborator

@blisc blisc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update changelog

…in script

Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
Signed-off-by: Yang Zhang <yangzhang@nvidia.com>
@yzhang123 yzhang123 requested review from ekmb and blisc February 20, 2020 20:58
@yzhang123 yzhang123 requested review from ekmb and removed request for ekmb February 20, 2020 21:06
@yzhang123 yzhang123 dismissed ekmb’s stale review February 20, 2020 21:35

handled all of issues

@yzhang123 yzhang123 merged commit 553b0ae into master Feb 20, 2020
@yzhang123 yzhang123 deleted the squad_inference_example branch February 20, 2020 21:35
dcurran90 pushed a commit to dcurran90/NeMo that referenced this pull request Oct 15, 2024
Closes NVIDIA#390 and NVIDIA#391

Signed-off-by: Trevor Grant <trevor.d.grant@gmail.com>
Signed-off-by: Mark Sturdevant <mark.sturdevant@ibm.com>
Co-authored-by: Mark Sturdevant <mark.sturdevant@ibm.com>
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.

3 participants