-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Azure.Core] Add Public Model Serialization #35665
Comments
Duplicate: #38763 |
Thanks for pointing to this tracking work item from the duplicated issues. It seems like the mile stone has been changed several times. Is this work item commited? Could you please also share an very rough ETA for this work item? |
The only statement that I can make is that this is something that we are working on. There is no committed timeline that I can share. |
Duplicate: #39107. |
Could you include clone methods? In our use cases, we need to adjust request options just before dispatching them. Modifying properties of the original instance is not convenient operation :) |
@m-nash, I believe the work on the Core side of this is complete with the release of System.ClientModel v1.0.0. If work remains to implement model serialization in various client libraries, should this be moved to track those separately? |
Find the tracking of this work very hard to follow. Items raised for specific models are closed as duplicates, referring to this item, e.g. #38763 (comment) But comment above suggests the core work is done and individual clients/models may need to add support, so should some of those "duplicates" be re-opened? Also the issue lists several issues which it is supposed to fix when done, but some of those are already closed, leaving the current state of this work unclear. Personally I'm interested the status of AnalyzeResult. |
Add Public Model Serialization classes to all Models
To be completed by May Milestone
Checklist:
azure-sdk-for-net/sdk/metricsadvisor/Azure.AI.MetricsAdvisor/src/Utf8JsonWriterExtensions.cs
Line 18 in 652dc91
Fixes:
ChatMessage
correctly? #35963The text was updated successfully, but these errors were encountered: