-
-
Notifications
You must be signed in to change notification settings - Fork 668
import error #605
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
Comments
@SkalaFrost, can you please provide me with the full code so I can try to test it? Also, did you follow this documentation? https://docs.praison.ai/docs/concepts/memory Thank you. |
Sure @MervinPraisonm blog_agent = Agent( Error: |
I get the same erroras SkalaFrost when trying to run the single agent example code. |
The praisonaiagents pip package is broken in latest versions (until 0.0.99 at least). Try:
and it will work. |
like memory, guardrail is also getting same import error |
@SkalaFrost @dev-anthonyrae @DracoBlue The bug has been fixed. Please upgrade to the latest version now and it should work if you want to use litellm config and memory then you need to use
Please let me know if you still have issues. Happy to review and fix it quickly. Here is the working code
Docs : How to use Models and config: https://docs.praison.ai/docs/models Note: I am also working on making the memory more efficient. So stay tuned. Thank you |
@sureshtmca can you please post your code you are running, then i could test it at my end. thank you |
@sureshtmca Please try this guardrail code Note : I include This is just an example
with multi agents
|
Same import error. Using
from praisonaiagents import Agent, PraisonAIAgents
from config import small_llm_config
# Create a simple agent
summarise_agent = Agent(
instructions="Summarise Photosynthesis",
llm=small_llm_config,
verbose=True,
)
# Run the agent
agents = PraisonAIAgents(agents=[summarise_agent])
agents.start()
By the way, version resolved by
|
@claude, review this error and propose a solution fix for this also in memory.py consider this, it should work both with litellm and without litellm Here is the change below
|
Claude finished @MervinPraison's task in 3m 14s —— View job • Memory Import Error Fix Complete ✅Todo List:
Summary of ChangesRoot Cause: The Memory class was properly defined but had dependency issues with OpenAI client imports that weren't handling optional dependencies correctly. Fix Applied:
Files Modified:
Testing:
The fix maintains backward compatibility while adding the requested litellm integration with proper fallbacks. Users should now be able to install and use praisonaiagents without import errors. |
@SkalaFrost @dev-anthonyrae @DracoBlue @sureshtmca @Ucag Please upgrade to the latest version and the issue has been fixed. If you still face issue, please raise the issue again or reopen this issue with the full code and output. Thanks for your support. |
Thanks @MervinPraison for the prompt fix! |
i could not keep calm without saying this. the structure of output generated from Agents and the validation of output (without user's much efforts) with instruction on reflection, is just just beautiful. You are awesome @MervinPraison |
I encountered an error while running the example.
Package version: 0.0.99
Error message:
ImportError: cannot import name 'Memory' from 'praisonaiagents.memory' (unknown location)
The text was updated successfully, but these errors were encountered: