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 English semantic search of pipelines #3718

Merged
merged 8 commits into from
Nov 16, 2022
Merged

Conversation

w5688414
Copy link
Contributor

PR types

  • New features

PR changes

  • Models&APIs

Description

  • Add English semantic search of pipelines with rocketqav2

@w5688414 w5688414 self-assigned this Nov 10, 2022
@w5688414 w5688414 added pipelines enhancement New feature or request labels Nov 10, 2022
@@ -96,6 +97,15 @@ def __init__(self,
assert (self.query_ernie is not None) or (self.title_ernie is not None), \
"At least one of query_ernie and title_ernie should not be None"

# Compatible to rocketv2 initialization
if (reinitialize):
Copy link
Collaborator

Choose a reason for hiding this comment

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

这块的语法有点奇怪,不需要加括号吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

wawltor
wawltor previously approved these changes Nov 16, 2022
Copy link
Collaborator

@wawltor wawltor left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@wawltor wawltor left a comment

Choose a reason for hiding this comment

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

LGTM

@w5688414 w5688414 merged commit cd29ed8 into PaddlePaddle:develop Nov 16, 2022
@w5688414 w5688414 deleted the pip40 branch November 1, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pipelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants