Skip to content

Radiology insights ga#2

Merged
koen-mertens merged 31 commits intomainfrom
radiology-insights-GA
Apr 19, 2024
Merged

Radiology insights ga#2
koen-mertens merged 31 commits intomainfrom
radiology-insights-GA

Conversation

@koen-mertens
Copy link
Owner

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

koen-mertens and others added 30 commits April 15, 2024 09:56
…zure#35056)

* Remove Optional type annotation from get() methods

* Remove duplicate overloads

* Update logging to include error category and message

* Revert change to jupyter notebook logging check

* Fix typing
* pin setuptools

* skip tests

* updates

* update
Moving away from SAS tokens for connecting to storage so switching to using a Workload Identity Federation connection to the container to download the needed files.

Co-authored-by: Wes Haggard <[email protected]>
* [Core][Corehttp] Update multipart test-server code

Aiohttp now doesn't set payload `Content-Length` headers. Let's stop
checking content length in the test server assertions.

Signed-off-by: Paul Van Eck <[email protected]>

* Update eng/tox/install_depend_packages.py

---------

Signed-off-by: Paul Van Eck <[email protected]>
Co-authored-by: Xiang Yan <[email protected]>
…re#35196)

* Add sanitizer enum and batch sanitizing method

* Add shared, fake credentials

* Add central set of sanitizers

* Remove KV custom matcher

* Add extra batch of sanitizers
* Show review link in logs and set DevOps project name based on the pipeline run

* Add project param in request

---------

Co-authored-by: Praveen Kuttappan <[email protected]>
With the common sanitizers added to test-proxy, several tests among our
SDKs are failing. This attempts to address a number of these failures.

Signed-off-by: Paul Van Eck <[email protected]>
* Create MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* Update MigrationGuide.md

* update

* Update MigrationGuide.md
* Update Readme

Fixed a typo that made the link to the concurrency sample not work.

* Update changelog

* Revert "Update changelog"

This reverts commit 1a7a6e8.
* Enable sampler for attach

* Allow sampler to take None param

* lint
Co-authored-by: azure-sdk <PythonSdkPipelines>
* make hint_policy common for all language

* support generate from tsp

* update

* fix python

* for debug

* small optimization

* enable trigger auto release with typespec folder

* add java assignee

* fix var name

* update excel

* change another folder

* fix env var

* small optimization

* optimize hint
* next-pylint azure-core

* next-pylint corehttp

* fix mypy

* apply azure-core change to corehttp
@koen-mertens koen-mertens requested a review from tomsft as a code owner April 19, 2024 06:42
@koen-mertens koen-mertens merged commit 8fc344b into main Apr 19, 2024
@koen-mertens koen-mertens deleted the radiology-insights-GA branch April 19, 2024 06:42
koen-mertens pushed a commit that referenced this pull request Aug 19, 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 #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 <[email protected]>

* Updates test.yml

* Uniquifier default to false for pipelines

* Updates from feedback

* Updates from feedback 2

---------

Co-authored-by: Graham Thomas <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
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.