Skip to content

Conversation

@fenilfaldu
Copy link
Contributor

📥 Pull Request

📘 Description

  • Added AutoGen instrumentation modules to support agent interactions, including agent-specific instrumentors and utilities.
  • Implemented new agent types and team structures for enhanced collaboration in AutoGen examples.
  • Updated documentation and examples to reflect the new features and usage scenarios, including a GroupChat and Swarm team example.

🧪 Testing
Tested Examples

@codecov
Copy link

codecov bot commented Jul 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Dwij1704
Copy link
Member

What happened to LangGraph? lol

@Dwij1704
Copy link
Member

I re ran it and it works

@fenilfaldu fenilfaldu requested review from Dwij1704 and dot-agi July 17, 2025 23:20
@fenilfaldu
Copy link
Contributor Author

So our integration tests are randomly failing on push, different ones each time.

What I suspect:
The validate_trace_spans() function (which I believe validates presence of LLM spans) is synchronous, while some of our examples involve async logic and await agent calls. There are chances the spans haven’t fully flushed/exported by the time validation kicks in.This might just be a race condition between span export and validation but this just all in theory for now.

@Dwij1704
Copy link
Member

So our integration tests are randomly failing on push, different ones each time.

What I suspect: The validate_trace_spans() function (which I believe validates presence of LLM spans) is synchronous, while some of our examples involve async logic and await agent calls. There are chances the spans haven’t fully flushed/exported by the time validation kicks in.This might just be a race condition between span export and validation but this just all in theory for now.

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.

Copy link
Member

@dot-agi dot-agi left a 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.

@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had any activity in the last 14 days.

If no updates are made within 7 days, this PR will be automatically closed.

@github-actions github-actions bot added the stale label Aug 19, 2025
@gitguardian
Copy link

gitguardian bot commented Aug 19, 2025

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
11016360 Triggered Generic Password 2ab3277 app/.github/workflows/test-api.yaml View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@github-actions github-actions bot removed the stale label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants