Adds LLM support to obtain the repository URL #386
Merged
Conversation
This file contains 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
|
|
lauraschauer
force-pushed
the
lascha-llm-support-for-repo-url
branch
from
47d60d2 to
d3cdfa6
Compare
LLM providers can be accessed through third party APIs (such as OpenAI), or through the Genrative AI Hub in the SAP AI Core.
lauraschauer
force-pushed
the
lascha-llm-support-for-repo-url
branch
from
4a8e466 to
375158d
Compare
copernico
reviewed
Jun 7, 2024
copernico
reviewed
Jun 7, 2024
copernico
reviewed
Jun 7, 2024
| attrs={ | ||
| "id": re.compile(r"ref-issue|ref-pullrequest"), | ||
| } |
There was a problem hiding this comment.
Not sure I understand what this change does, could you explain briefly?
There was a problem hiding this comment.
I did actually make a change here, I added line 142: if content is not None
The reason is that if the request in fetch_url() fails to get the content of a webpage, it logs an error and returns Null. If this isn't checked in this extract_ghissue_references() function, the program crashes when extract_ghissue_references() tries to do content.find_all(), because in this case it would be None.find_all().
copernico
reviewed
Jun 7, 2024
…backend magic strings to constants
…can be called with this model. This is because only one instantiation of the model is needed throughout the whole runtime of prospector
lauraschauer
force-pushed
the
lascha-llm-support-for-repo-url
branch
from
040fa79 to
0a09951
Compare
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.
This adds code to allow users to optionally use LLM support. If they do, it is not necessary to supply the repository URL as an argument to Prospector. Instead, it will automatically be obtained through the chosen LLM provider.