-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[BUG] DuckDuckGoSearchRM is not working - AttributeError #231
Comments
Sorry for the delay. Could you check whether the rm can output content with some queries written by yourself? Given you are using a quantized model, the model may fail to output correct queries. |
The RM works fine, after some more testing I narrowed the issue down to the LLMs. I tested Ollama models in a up to 80GB VRAM range (both quantized and full-precision). The only model that didn't run into errors was Llama 3.2 3b (fp), however the output was only the introduction section. I also used the DSPy templates from the ollama example. Have you found success with a model from the Ollama model library? If so, which one(s) was it? Tested LLMs:
|
I had to go through multiple attribute errors but finally got the script to work. I've submitted a pull request. Even if you don't use my specific script, it'll likely contain updated logic to handle the semi-recent changes to the duckduck_go dependency. See here: |
Describe the bug
When using DuckDuckGoSearchRM in the STORMWikiRunner, I always get an attribute error in STORMWikiRunner.run(...): 'AssertionError' object has no attribute 'message'. The underlaying error comes from
_text_api
function in theDDGS
class in the duckduckgo_search library, fileduckduckgo_search.py
:AssertionError: keywords is mandatory
.I used a LLM from Ollama:
llama3.1:8b-instruct-fp16
.To Reproduce
See 'Code' section below
Environment:
Code
Error Message
I also tried GoogleSearch instead of DuckDuckGoSearchRM, which logs a lot of errors in the search phase but it runs through however.
The code is the same, I only adjusted the
rm
variable:Generated output:
generated.zip
Google Search Error Logs
The text was updated successfully, but these errors were encountered: