[CODEOWNERS] Update codeowners entry for Contoso Widget Manager #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) * Handle tool/directory copy better * Use /usr/bin/env instead of /bin/env * Append additional newline --------- Co-authored-by: Ben Broderick Phillips <[email protected]>
* Install azsdk mcp server in copilot setup steps * use pwsh shell
…e#42174) Signed-off-by: Paul Van Eck <[email protected]>
…e#42220) * add parameter to skip changelog generation if needed * update
…wner) (Azure#42237) * code and test * update changelog * Update CHANGELOG.md --------- Co-authored-by: azure-sdk <PythonSdkPipelines> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: ChenxiJiang333 <[email protected]>
* session container fixes * async changes * sync changes * mypy/pylint * Update _session.py * mark query plan as fetched for query * adjust logic after merging * Update _base.py * Update _base.py * tests - missing sync mwr * sync mwr tests, test fixes * Update test_session_async.py * Update test_backwards_compatibility.py * Update test_backwards_compatibility.py * Update test_backwards_compatibility_async.py * Update test_backwards_compatibility_async.py * Update test_backwards_compatibility.py * Update execution_dispatcher.py * merge leftovers * slip * Update test_backwards_compatibility.py * Update test_backwards_compatibility.py * Update test_query_hybrid_search_async.py * further changes, changelog * add tests * typehint * address comments * Update _session.py * Update _base.py * ci tests * merging main * Update _cosmos_client_connection.py * tests * change session token logic * Update _session.py * small fixes * Update _session.py * Update _session.py * updates * update tests that checked for compound tokens - this will no longer be the case * Update test_config.py * Update execution_dispatcher.py * remove query logic * remove partition split testing * oylint * Update dev_requirements.txt * delete duplicate * update tests for partition split * change timeout of split partition key tests reduces timeout from 25 minutes to 7 minutes * update test * Test fixes Fixes tests to accomodate the extra readfeed that may happen during a read item operation. This also includes other general test fixes. * update tests * update tests Updates tests and fixes a key error issue in session.py * add logger to session token * Update _session.py * Update _session.py * Update test_session.py * async changes * small test fixes - align sync and async * small updates * Update documents.py * test updates * Update _session.py * async tests * pylint * tests * logic fixes, test updates * leftover changes * match session logic exactly * propagate headers to request params for retry logic * Update CHANGELOG.md * revert session token dropping * last test updates * Update _session.py * update session token on errors * reformat internal exceptions * change string for constant * address comments --------- Co-authored-by: Simon Moreno <[email protected]>
Signed-off-by: Paul Van Eck <[email protected]> Co-authored-by: Copilot <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
* Update OTel and remove Python 3.8 * changelog * Update sdk/monitor/azure-monitor-opentelemetry/CHANGELOG.md Co-authored-by: Copilot <[email protected]> * Fix resource detector version --------- Co-authored-by: Copilot <[email protected]>
) * Use full checkout for regen stage * remove fetchdepth * update both * checkout branch * switch * Fetch branch * origin * azure-sdk --------- Co-authored-by: jolov <[email protected]>
* Fix update_agents * Update sdk/ai/azure-ai-agents/tests/test_agents_mock_overloads.py Co-authored-by: Copilot <[email protected]> * Update sdk/ai/azure-ai-agents/CHANGELOG.md Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
MetricsClient and MetricsQueryClient have been removed from the azure-monitor-query package. This is part of the migration to split logs and metrics functionality into separate packages and using TypeSpec as the generation tooling for this SDK. - The MetricsClient has been moved to the new azure-monitor-querymetrics package. - Functionality provided by MetricsQueryClient can be access through the azure-mgmt-monitor package. - All metrics-related content has been removed. - The package version has been updated to 2.0.0 due to the breaking changes. Signed-off-by: Paul Van Eck <[email protected]>
* update sample and retry documentation * react to comments * fix analyze
Azure#42267) * code and test * update changelog * update changelog --------- Co-authored-by: azure-sdk <PythonSdkPipelines> Co-authored-by: ChenxiJiang333 <[email protected]>
…remove promptflow) (Azure#42243) * chore: Remove promptflow dependencies * fix: Filter inputs before calling function in Batch Engine Added to match behavior in promptflow see also: https://github.com/microsoft/promptflow/blob/5e6c183474c0a2575bb416d18201e4f9fd562b2e/src/promptflow-core/promptflow/executor/_script_executor.py#L162 * refactor: Use enumerate instead of manually keeping track of line number * fix,refactor: Unconditionally inject default column mapping from data -> params In promptflow's logic for applying a column mapping to data, it will unconditionally inject a mapping from function parameter do data of the same name: https://github.com/microsoft/promptflow/blob/3e297112a2c142caf7c185bcba644d0f66422539/src/promptflow-devkit/promptflow/batch/_batch_inputs_processor.py#L110-L141 This behavior deviated from the existing logic in this SDK, where generating that mapping was conditional on the user not providing a column mapping: https://github.com/Azure/azure-sdk-for-python/blob/f3740540eb5b3d22dc1bccba0eb00b652b124d5f/sdk/evaluation/azure-ai-evaluation/azure/ai/evaluation/_legacy/_batch_engine/_run_submitter.py#L51-L52 This deviation caused one of the parameterized cases of `test_evaluate_another_questions` to fail, because there was a user provided column mapping that mapped to a parameter not present in the evaluator, so the lack of a default mapping caused the evaluation to fail because the required parameter was missing https://github.com/Azure/azure-sdk-for-python/blob/f3740540eb5b3d22dc1bccba0eb00b652b124d5f/sdk/evaluation/azure-ai-evaluation/tests/e2etests/test_evaluate.py#L216 This commit aligns the application of the column mapping in the SDK more closely to the promptflow implementation * refactor: Don't shadow `value` variable in apply_column_mapping_to_lines * feat: Add support for running aggregations in RunSubmitterClient * tests,fix: Don't log duration as a metric Breaks a tests that checks for strict equality of metrics * refactor: Rewrite RunSubmitterClient.get_details * fix: Correct the typing of is_onedp_project * fix,tests: Don't log tokens as metrics Removing to match the behavior of the other clients * fix: Set error message without depending on run storage Promptflow surfaces exceptions by reading them from their "Storage" abstraction. That has not been ported to this SDK. * tests,fix: Fix test_evaluate_invalid_column_mapping PR 40556 accidentally indented the assertion in test_evaluate_invalid_column_mapping into the `pytest.raises` block. This inadvertently made the test useless, since the `evaluate` call would always raise an exception which skips over the assertion as the exception unwinds the stack. This commit unindents the assertion so that it runs. Additionally, PR 41919 updated our validation logic to allow column mapping reference of arbitrary length e.g. `${target.foo.bar.baz`}`. So this commit also removes the test case that was explicitly guarding against this `${target.response.one}` * fix,tests: Force PFClient specific tests to use PFClient * fix,tests: Force CodeClient specific tests to use CodeClient * fix: Improve the ergonomics for picking which client is used Except for the CodeClient, you only need to use at most 1 of `_use_pf_client` and `_use_run_submitter_client` * feat: Show exception message in run logs * fix: For safety evaluation to use codeclient as originally intended * fix: Don't wrap EvaluationExcpeiton in BatchEngineError * refactor: Refactor BatchConfig * fix: Make raising on error configurable for runsubmitterclient * chore: Update changelog * fix: Uncomment log_path * chore: Add promptflow to dev-requirements.txt Some tests have explicit dependencies on the promptflow implementation. Since we aren't removing the code path yet, allow them to run by installing promptflow for those tests. * fix: Initialize error_message * fix: Get exception instead of batchrunresult * chore,docs: Clarify changelog
* Check for pyproject.toml * Only attempt pyproject parsing if setup.py fails * Add test and documentation * Pass lib_dir to from_path instead of file path * Updated parsing exception
Migration guide added for the 2.0.0 azure-monitor-query library. Signed-off-by: Paul Van Eck <[email protected]> Co-authored-by: Scott Addie <[email protected]>
- Update changelog for release - Rename Azure AD references with Microsoft Entra ID. Signed-off-by: Paul Van Eck <[email protected]>
This adds the `azure-sdk-write-keyvault` team as code owners, removing `lgonsoulin`.
* Add Check class to be used as common definition for a check * add whl and import_all check entrypoints * refactor to allow automatic venv isolation * update readme Co-authored-by: Laurent Mazuel <[email protected]> Co-authored-by: Scott Beddall <[email protected]>
* Add Powershell Installation Instructions for Copilot * Update copilot-instructions.md * more specific
…t/tspconfig.yaml', API Version: 1.0.0-beta.1, SDK Release Type: beta, and CommitSHA: 'ac0c4053b00c71d590c6d8ba7f7b0fc78131850a' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5264549 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
Added a test comment to the CHANGELOG.
github-actions
bot
added
AgriFood
AI Agents
AI Model Inference
AI Projects
Cognitive - Anomaly Detector
App Configuration Provider
App Configuration
Attestation
Batch
Cognitive - Language
CognitiveServices
Communication - Call Automation
Communication - Common
Communication - Chat
Communication - Email
Communication - Identity
Communication - Job Router
Communication - Messages
Communication - Phone Numbers
Communication - Rooms
Communication - SMS
Confidential Ledger
Container Registry
labels
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update codeowners entry for Contoso Widget Manager