Skip to content

Add types from AIJsonUtilities.JsonContext to reporting JsonContext - #6078

Merged
peterwald merged 5 commits into
mainfrom
pewaldsc/serializer
Mar 12, 2025
Merged

Add types from AIJsonUtilities.JsonContext to reporting JsonContext#6078
peterwald merged 5 commits into
mainfrom
pewaldsc/serializer

Conversation

@peterwald

@peterwaldpeterwald commented Mar 11, 2025•
edited by dotnet-policy-service Bot
Loading

Copy link
Copy Markdown
Member
Microsoft Reviewers: Open in CodeFlow

@peterwaldpeterwald added the area-ai-eval Microsoft.Extensions.AI.Evaluation and related label Mar 11, 2025
@peterwaldpeterwald self-assigned this Mar 11, 2025
@peterwald
peterwald requested a review from a team as a code ownerMarch 11, 2025 18:24
@shyamnamboodiripadshyamnamboodiripad mentioned this pull request Mar 11, 2025
Merged
shyamnamboodiripad
shyamnamboodiripad approved these changes Mar 12, 2025
View reviewed changes

@shyamnamboodiripadshyamnamboodiripad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏾 modulo comments

@peterwald
peterwald enabled auto-merge (squash) March 12, 2025 20:37
@peterwald
peterwald merged commit ab01355 into mainMar 12, 2025
@peterwald
peterwald deleted the pewaldsc/serializer branch March 12, 2025 21:01
peterwald added a commit that referenced this pull request Mar 13, 2025
@peterwald
Add types from AIJsonUtilities.JsonContext to reporting JsonContext (#…
077d4f3
…6078)
* Add types from AIJsonUtilities.JsonContext to reporting JsonContext
* Add a few more tests
* Chain M.E.AI.Eval serialization types through AIJsonUtilities
* Fix reversed Compact/Default
* Rename some of the types.
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Apr 12, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@stephentoubstephentoubstephentoub left review comments
@eiriktsarpaliseiriktsarpaliseiriktsarpalis left review comments
@shyamnamboodiripadshyamnamboodiripadshyamnamboodiripad approved these changes
+1 more reviewer
@RussKieRussKieRussKie left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@peterwaldpeterwald

Labels

area-ai-evalMicrosoft.Extensions.AI.Evaluation and related

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@peterwald@stephentoub@eiriktsarpalis@RussKie@shyamnamboodiripad