-
Notifications
You must be signed in to change notification settings - Fork 3
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
Multistep search #399
Multistep search #399
Conversation
I think this fits the idea very well apart from how the user triggers it. Ideally the user should have two Boolean arguments:
the first one will prepend a transfer learning to the main search, the second one will append the MBR search to the main search. The config specification of what should be overwritten looks good. |
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.
LGTM
version 1:
update 1:
update 2:
This PR adds the multistep search including tests, UI, docs, and e2e tests.