Skip to content

Add interactive search script#99

Draft
amaswea wants to merge 9 commits intomainfrom
aswearngin/port-interactive-search
Draft

Add interactive search script#99
amaswea wants to merge 9 commits intomainfrom
aswearngin/port-interactive-search

Conversation

@amaswea
Copy link
Copy Markdown
Contributor

@amaswea amaswea commented Oct 9, 2025


PR Checklist (do not remove):

  • I've added necessary new tests and they pass
  • My PR description explains how to test this contribution
  • I have linked this PR to an issue
  • I have requested reviews from two people

@willeppy
Copy link
Copy Markdown
Collaborator

@amaswea do we still want to merge this or can I close for now then we can revisit in future if we want?

@amaswea
Copy link
Copy Markdown
Contributor Author

amaswea commented Oct 22, 2025

@amaswea do we still want to merge this or can I close for now then we can revisit in future if we want?

@willeppy This was primarily for testing purposes so that we could be more sure of the otuputs of the search implementation. I don't think we need it for the open source release, but it might be useful for providing an example of how we could add human-in-the-loop component to the marketplace. So maybe we could close it, but would want to make sure we can easily bring it back / don't forget about it in the case we want to do something similar.

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.

2 participants