Open
Conversation
… added comment in env example about llm token usage
|
Can one of the admins verify this patch? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Guidelines for Pull Requests
If you haven't yet read our code review guidelines, please do so, You can find them here.
Please confirm the following by adding an x for each item (turn
[ ]into[x]).Please provide a brief description of your ticket
The AI will now return structured information about search results, those need to be shonw as a results list
Description
{
"search_results":[
{
"document_id":"12",
"url":"https://www.cdc.gov/infection-control/hcp/core-practices/index.html",
"title":"CDC's Core Infection Prevention and Control Practices for Safe Healthcare Delivery in All Settings | Infection Control | CDC",
"summary":"Covers standard precautions, transmission-based precautions, infection control practices, and PPE use for managing infectious disease outbreaks in healthcare settings. Includes guidelines for early detection, respiratory hygiene, and facility-wide protocols during high community transmission periods.",
"relevance":"Directly addresses hospital preparedness for infectious disease outbreaks through core infection prevention practices, standard precautions, and response protocols critical for healthcare facilities."
},
{
"document_id":"10",
"url":null,
"title":"Preparedness for the deliberate use of biological agents",
"summary":"Discusses planning for bioterrorism events, emphasizing treating all outbreaks as natural until proven otherwise, public health worker roles in early detection, training for biological casualties, and surge capacity limitations during large-scale incidents.",
"relevance":"Relevant to hospital preparedness for deliberate biological threats, highlighting protocols for outbreak response, resource allocation, and coordination with public health systems."
}
]
}
Hopper needs to be reworked so it behaves just as a document search engine. It should show the found documents with a link to them (either url or link to the pdf that should have been uploaded into Hopper) with the title, summary, and relevance shown.
If there are no search results, it should just say “No documents were found.”.
HOP-56
Anything else the reviewer needs to know?
Does this Pull request depend on any other?