-
Notifications
You must be signed in to change notification settings - Fork 456
introduce AutoGen instrumentation. #1157
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
What happened to LangGraph? lol |
I re ran it and it works |
So our integration tests are randomly failing on push, different ones each time. What I suspect: |
Few examples are failing due to backend failure, o3 example is busted, @areibman pushed it yesterday as it was working on his machine, will need to check what the actual reason is behind its failure. ag2, web_search and mem0 were already busted, needs fixing as discussed yesterday. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments for improvements.
AutoGen example notebooks are not clean at all. They are copypasta from the Python scripts. I cannot stress the importance enoughof having clean and structured notebooks as they provide the users with running examples out of the box.
Please look at the other notebooks and structure the content properly.
📥 Pull Request
📘 Description
🧪 Testing
Tested Examples