Add dependancy injection to Tools #320

davorrunje · 2024-12-30T17:40:15Z

Why are these changes needed?

Related issue number

Checks

I've included any doc changes needed for https://docs.ag2.ai/. See https://docs.ag2.ai/docs/contributor-guide/documentation to build and test documentation locally.
I've added tests (if relevant) corresponding to the changes introduced in this PR.
I've made sure all auto checks have passed.

…o dependency_injection.py

…dantic_field function

Co-authored-by: Davor Runje <[email protected]>

…dency-injection-rj

Implement dependancy injection as defined in PR 320

davorrunje and others added 5 commits December 30, 2024 10:23

initial version

4341b49

Dependency injection POC notebook wip

10d901b

Dependency injection POC notebook wip

e0e2cb1

Dependency injection POC notebook completed

d0b968b

Initial dependency injection implementation

e037a9b

davorrunje self-assigned this Dec 30, 2024

rjambrecic and others added 6 commits December 31, 2024 09:24

Dependency injection with annotated parameters WIP

2523aac

function_utils.py moved into tools

d1def3d

Move remove_injected_params_from_signature and annotation_context int…

ced9e83

…o dependency_injection.py

Replace annotation_context context manager with string_metadata_to_py…

07e6f5c

…dantic_field function

Fix tests WIP

4cad8f2

Fix tests WIP

aa449f7

davorrunje marked this pull request as draft December 31, 2024 17:34

rjambrecic and others added 17 commits January 2, 2025 10:11

Limit fast_depends version < 3

14e1b2e

Co-authored-by: Davor Runje <[email protected]>

Crate inject_params function and use it eithin conversable_agent.py

98ca1dd

Set asyncer to fix version

d98c3ed

Co-authored-by: Davor Runje <[email protected]>

Add async tests for dependency injection

dd56413

Rename DescriptionField to Field

6d2305d

Add support for Depends(f)

2b9dd9b

Dependency injection tests refactoring

72d81cc

Add support for Depends with default value assigned

7df688f

Update tests

5e4c8b1

Merge remote-tracking branch 'origin/main' into dependency-injection-rj

0aa0dd3

Merge remote-tracking branch 'origin/main' into dependency-injection

ad27346

Merge remote-tracking branch 'origin/dependency-injection' into depen…

c4e61aa

…dency-injection-rj

Initial dependency injectio notebook added

010c999

Fix failing pre-commit

d5588f0

refactoring

dbb9e56

Add dependency injection tests

6252778

Merge pull request #330 from ag2ai/dependency-injection-rj

e401fb5

Implement dependancy injection as defined in PR 320

davorrunje requested a deployment to openai1 January 3, 2025 14:24 — with GitHub Actions Waiting

davorrunje requested review from sternakt, rjambrecic and davorinrusevljan January 3, 2025 14:44

sternakt approved these changes Jan 3, 2025

View reviewed changes

davorrunje added this pull request to the merge queue Jan 3, 2025

github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jan 3, 2025

merge with main

4e66c83

davorrunje requested a deployment to openai1 January 3, 2025 14:55 — with GitHub Actions Waiting

davorrunje enabled auto-merge January 3, 2025 14:55

update conftest import

b1f16ac

davorrunje deployed to openai1 January 3, 2025 15:08 — with GitHub Actions Active

davorrunje had a problem deploying to openai1 January 3, 2025 15:08 — with GitHub Actions Failure

davorrunje temporarily deployed to openai1 January 3, 2025 15:09 — with GitHub Actions Inactive

davorrunje added this pull request to the merge queue Jan 3, 2025

Merged via the queue into main with commit 3c33a1b Jan 3, 2025
228 of 229 checks passed

davorrunje had a problem deploying to openai1 January 3, 2025 15:31 — with GitHub Actions Failure

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add dependancy injection to Tools #320

Add dependancy injection to Tools #320

davorrunje commented Dec 30, 2024 •

edited

Loading

Add dependancy injection to Tools #320

Add dependancy injection to Tools #320

Conversation

davorrunje commented Dec 30, 2024 • edited Loading

Why are these changes needed?

Related issue number

Checks

davorrunje commented Dec 30, 2024 •

edited

Loading