Skip to content
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

Update my main #2

Merged
merged 2,144 commits into from
Sep 10, 2024
Merged

Update my main #2

merged 2,144 commits into from
Sep 10, 2024

Conversation

iscai-msft
Copy link
Owner

No description provided.

mccoyp and others added 30 commits July 31, 2024 16:39
* removing experimental attribute from connection entity

* removed unused import stmt
* fixing sweep job service response parsing issue

* fixing sweep job service response parsing issue

* raising a work around for cmd job and spark job name lowering case

* fixing black issue

* fixing serialization issue in sweep job

* include black fix

* found and fix the issue in compute validation

* fixing a corrupt wokspace arm template

* fixing the failed test case

* updating the date for 1.18.0 release

* removing empty section

* adding in progress version details

* adding IDC code reviewers

* correcting the directory path

* lint fix

* adding change log entry

* addressing review comment

* fixing ut

* addressing api review comment
azure-identity already requires msal>=1.29.0, so this should be safe.

Fixes Azure#36713
* optimize changelog output

* open breaking change detection for mgmt sdk

* adopt breaking change detector

* format

* update

* update

* update

* update for requirements

* Update package_utils.py
* update typespce-python

* update typespce-python
* generate new REST client

* implement MQTT Generate Token

* add test case

* ignore check spell words

* update changelog

* update changelog

* update latest version

* update changelog

* add EOL

* change field name to webpubsub_client_access

* rename to webpubsub_client_protocol

* rename to client_protocol

* update changelog

---------

Co-authored-by: chuongnguyen <chuongnguyen@microsoft.com>
* optimize log

* add hint message

* update stderr

* update log format

* update log format

* update hint message

* update

* update
* fixing sweep job service response parsing issue

* fixing sweep job service response parsing issue

* raising a work around for cmd job and spark job name lowering case

* fixing black issue

* fixing serialization issue in sweep job

* include black fix

* found and fix the issue in compute validation

* fixing a corrupt wokspace arm template

* fixing the failed test case

* updating the date for 1.18.0 release

* removing empty section

* adding in progress version details

* adding IDC code reviewers

* correcting the directory path

* lint fix

* adding change log entry

* addressing review comment

* fixing ut

* addressing api review comment

* adding a fix for subnet validation in compute instance

* my py fix
* Update tsp-location.yaml with latest commit id

* regenerated SDK
.\venv\Scripts\activate
pip install -r dev_requirements.txt
pip install tox<5
pip install -e .
adjustments sample_critical_result_inference_async

* inference

* Adjusted README.md

* adjustment tests
fixes pylint

* fix get result call on inferences inside job

* implementation RadiologyInsightsInferenceResult in samples and tests

* removed headers from test files

* cspell updates & fix typos

Update cspell file & fix some unknown words for typos

* fix tests in record mode

* fix spelling errors

* fix spelling error

* Update sample_finding_inference.py

* Update sample_finding_inference.py

* fix reviewer comments

* add breaking changes to changelog and fix pylint issue

* Update assets.json

* Regenerated SDK for authentication, adjusted samples

* Update CHANGELOG.md

* fixing PR conversations

* small cosmetic changes

* Update README.md

* refactor sample and adjust changelog

* Update sample_age_mismatch_inference_async.py

* adjust async samples according to review

* Update README.md

* fix tests for other packages

* Update assets.json

* Update assets.json

* Update assets.json

* regenerated SDK

* disable healthinsights tests

* disable cancerprofiling and trialmatcher test

* disable tests

* remove key based authentication from samples and documentation

* fix tests

* push recordings to assets

* change sync to async cred

* remove credentials sample and update documentation

* Update sdk/healthinsights/azure-healthinsights-radiologyinsights/README.md

Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com>

* Update sdk/healthinsights/azure-healthinsights-radiologyinsights/README.md

Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com>

* change samples and add a patch method

* adjust tests

* fix tests for radiology insights

* - adjust def documentation in patch.py
- remove unused imports in patch.py
- resolve duplicate references in models.py
- resolve duplicate and wrong references in README.md

* - fix exception handling in samples
- move imports in samples
- fix pylint and mypy issues in patch files

* remove class from sync samples
replace deserialize with job iso inference result

* - fix bugs sample_finding_inference.py
- fix bugs sample_laterality_discrepancy_inference.py
- adjust _operations

* undo changes in patch files :
replace deserialize Job with InferenceResult

* - revert changes in _operations
- replace with expected code in _patch

* postprocess changes

* revert _patch

* fix pylint error in _operations.py
fix mypy error in _patch.py (add ignore)

* move ignore comment in _patch.py
adjust sample_radiology_procedure_inference to align with the other samples
adjust sample_radiology_procedure_inference_async to align with the other samples
adjust sample_sex_mismatch_inference to align with the other samples
adjust sample_sex_mismatch_inference_async to align with the other samples

---------

Co-authored-by: cbovijn <51367245+cbovijn@users.noreply.github.com>
Co-authored-by: Celia Bovijn <celiabovijn@microsoft.com>
Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com>
* Remove Optional type annotation from get() methods

* Remove duplicate overloads

* Update CODEOWNERS to IDC only

* Revert unintended output change

* Fix typo

* Simplify routing
* update targeted proxy version

---------

Co-authored-by: Scott Beddall <scbedd@microsoft.com>
Reduces flakiness

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
Since find is deprecated, search is the API we should be using.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
* fix issue33658

* fix issue33658 2
The `Az.Accounts` module for PowerShell will soon start outputting
token data as secure strings. This ensures that `AzurePowerShellCredential`
can handle this format.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
* sync side fix for rotate primary key issue in hub

* sync uamqp transport

* add to async

* remove print/cleanup

* mypy/pylint

* changelog

* fix logging formatting

* no retry on all auth exceptions

* remove import error in uamqp recv, b/c no external dependency

* mypy
… SDK owner) (Azure#36723)

* code and test

* update-testcase

* Update testcases

* Update CHANGELOG.md

---------

Co-authored-by: azure-sdk <PythonSdkPipelines>
Co-authored-by: ChenxiJiang333 <v-chenjiang@microsoft.com>
Co-authored-by: ChenxiJiang333 <119990644+ChenxiJiang333@users.noreply.github.com>
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
cn0238 and others added 23 commits September 6, 2024 17:15
* fixing header bug

* Async Discover + Failover

* Fixing Test, mypy, and pylint issues

* Fix test async issue

* review comments

* more spelling fixes

* Trying to fix async test issue

* try to fix async issue

* Update test_configuration_async_client_manager.py

* async discovery

* Rename, add sync lock, add type check

* removing unneeded lock.

* client base

* Added missing lock
* [SchemaRegistry] handle unknown content types

* remove extra space in Accept headers

* fix license in tests

* fix spacing in accept header async

* add tests with mocked unknown content type

* mypy/lint

* cspell
The version of Azure CLI no longer needs to be pinned. Removing this pin
should also resolve the CI failures trying to install it.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
Co-authored-by: Catalina Peralta <caperal@microsoft.com>
Azure#37048)

* code and test

* Update CHANGELOG.md

* Update CHANGELOG.md

* Delete sdk/sql/azure-mgmt-sql/assets.json

* update-testcases

* Update CHANGELOG.md

---------

Co-authored-by: azure-sdk <PythonSdkPipelines>
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
Co-authored-by: ChenxiJiang333 <119990644+ChenxiJiang333@users.noreply.github.com>
Co-authored-by: ChenxiJiang333 <v-chenjiang@microsoft.com>
Co-authored-by: James Suplizio <jasupliz@microsoft.com>
* Refactor prepare-pipelines

* Convert pipeling generation single to use template

* Convert to pwsh for setting the variables

* Fix switch expression

* Add path separator in front of devopspath

* Continue on error for individual step failures

* Remove unnecessary parameters

---------

Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
…xtended template (Azure#36994)

* update build_extended job to direct component detection to sdk/<servicedirectory>
* Provider Release Update

* Updated docs and readme

* Update README.md

* giving replica_client_manager a default value so it's not a breaking change.

* re-write to not have a breaking change.

* Update README.md
* Making change unbreaking for API

* removing endpoint
* update release date

* update release date
… owner) (Azure#37271)

* code and test

* Update _version.py

* Update CHANGELOG.md

---------

Co-authored-by: azure-sdk <PythonSdkPipelines>
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
@iscai-msft iscai-msft merged commit 5bb372f into main Sep 10, 2024
@iscai-msft iscai-msft deleted the update_my_main branch September 10, 2024 15:42
iscai-msft pushed a commit that referenced this pull request Sep 10, 2024
…e#36041)

* Initial commit of Health.Deidentification dataplane

* Use MI instead of SAS

* Regenerates with Plaintext

* Adds rest of tests

* First attempt patch

* Patch Attempt #2

* Patch Attempt Azure#3

* Creates base recordings

* Fixes sanitizers; Test replay functioning

* Creates all sync samples

* Creates all async samples

* Adds description in readme

* Adds tsplocation

* Checkpoint

* Executes test recording migration

* Adds pipeline yamls

* Updates ci.yml triggers

* Removes ArtifactName from ci.yaml

* Fixes analysis failures

* Fixes analysis failures 2

* Update sdk/healthdataaiservices/ci.yml

Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>

* Updates test.yml

* Uniquifier default to false for pipelines

* Updates from feedback

* Updates from feedback 2

---------

Co-authored-by: Graham Thomas <gthomas@microsoft.com>
Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.