Skip to content

Commit f439746

Browse files
alliscodeCopilot
andcommitted
Fix pyright: remove phantom FoundryMemoryProvider import, apply ruff format
- Remove import of non-existent _foundry_memory_provider module (incorrectly kept during rebase conflict resolution) - Apply ruff formatter to test_local_eval.py and get-started samples Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 8a2c237 commit f439746

5 files changed

Lines changed: 22 additions & 6 deletions

File tree

python/packages/azure-ai/agent_framework_azure_ai/__init__.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
evaluate_foundry_target,
3030
evaluate_traces,
3131
)
32-
from ._foundry_memory_provider import FoundryMemoryProvider
3332
from ._project_provider import AzureAIProjectAgentProvider # pyright: ignore[reportDeprecated]
3433
from ._shared import AzureAISettings
3534

@@ -62,7 +61,6 @@
6261
"AzureTokenProvider",
6362
"AzureUserSecurityContext",
6463
"FoundryEvals",
65-
"FoundryMemoryProvider",
6664
"RawAzureAIClient",
6765
"RawAzureAIInferenceEmbeddingClient",
6866
"__version__",

python/packages/core/tests/core/test_local_eval.py

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1037,11 +1037,17 @@ def test_all_passed_ignores_own_counts_when_none(self):
10371037
from agent_framework._evaluation import EvalResults
10381038

10391039
sub_pass = EvalResults(
1040-
provider="Local", eval_id="e1", run_id="r1", status="completed",
1040+
provider="Local",
1041+
eval_id="e1",
1042+
run_id="r1",
1043+
status="completed",
10411044
result_counts={"passed": 2, "failed": 0, "errored": 0},
10421045
)
10431046
parent = EvalResults(
1044-
provider="Local", eval_id="e0", run_id="r0", status="completed",
1047+
provider="Local",
1048+
eval_id="e0",
1049+
run_id="r0",
1050+
status="completed",
10451051
result_counts=None,
10461052
sub_results={"agent1": sub_pass},
10471053
)
@@ -1052,11 +1058,17 @@ def test_all_passed_parent_fails_when_own_counts_fail(self):
10521058
from agent_framework._evaluation import EvalResults
10531059

10541060
sub_pass = EvalResults(
1055-
provider="Local", eval_id="e1", run_id="r1", status="completed",
1061+
provider="Local",
1062+
eval_id="e1",
1063+
run_id="r1",
1064+
status="completed",
10561065
result_counts={"passed": 2, "failed": 0, "errored": 0},
10571066
)
10581067
parent = EvalResults(
1059-
provider="Local", eval_id="e0", run_id="r0", status="completed",
1068+
provider="Local",
1069+
eval_id="e0",
1070+
run_id="r0",
1071+
status="completed",
10601072
result_counts={"passed": 1, "failed": 1, "errored": 0},
10611073
sub_results={"agent1": sub_pass},
10621074
)

python/samples/01-get-started/02_add_tools.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ def get_weather(
2727
"""Get the weather for a given location."""
2828
conditions = ["sunny", "cloudy", "rainy", "stormy"]
2929
return f"The weather in {location} is {conditions[randint(0, 3)]} with a high of {randint(10, 30)}°C."
30+
31+
3032
# </define_tool>
3133

3234

python/samples/01-get-started/04_memory.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,8 @@ async def after_run(
5959
text = msg.text if hasattr(msg, "text") else ""
6060
if isinstance(text, str) and "my name is" in text.lower():
6161
state["user_name"] = text.lower().split("my name is")[-1].strip().split()[0].capitalize()
62+
63+
6264
# </context_provider>
6365

6466

python/samples/01-get-started/05_first_workflow.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ def create_workflow():
4545
"""Build the workflow: UpperCase → reverse_text."""
4646
upper = UpperCase(id="upper_case")
4747
return WorkflowBuilder(start_executor=upper).add_edge(upper, reverse_text).build()
48+
49+
4850
# </create_workflow>
4951

5052

0 commit comments

Comments
 (0)