Skip to content

Update Analytics to use new API #6221

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Jul 23, 2025
Merged

Update Analytics to use new API #6221

merged 12 commits into from
Jul 23, 2025

Conversation

maryamziaa
Copy link
Contributor

@maryamziaa maryamziaa commented Jul 16, 2025 •
edited
Loading

Proposed change(s)

The old analytics API has been deprecated. This PR updates the analytics implementation to use the new API.

Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)

Types of change(s)

  • Bug fix
  • New feature
  • Code refactor
  • Breaking change
  • Documentation update
  • Other (please describe): Upgraded to new Analytics API

Checklist

  • Added tests that prove my fix is effective or that my feature works
  • Updated the changelog (if applicable)
  • Updated the documentation (if applicable)
  • Updated the migration guide (if applicable)

Other comments

maryamziaa
maryamziaa commented Jul 16, 2025
View reviewed changes
maryamziaa
maryamziaa commented Jul 17, 2025
View reviewed changes
@maryamziaa maryamziaa requested a review from a team July 17, 2025 21:57
antoine-agthe-unity
antoine-agthe-unity approved these changes Jul 18, 2025
View reviewed changes
Copy link

@antoine-agthe-unity antoine-agthe-unity left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I invite you to give more explanation about why IAnalytics and IData types are implemented that way as it might look exotic, who an external point of view.

@maryamziaa maryamziaa merged commit 8034047 into develop Jul 23, 2025
23 checks passed
@maryamziaa maryamziaa deleted the analytics-upgrade branch July 31, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexander-suvorov alexander-suvorov alexander-suvorov approved these changes

@antoine-agthe-unity antoine-agthe-unity antoine-agthe-unity approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maryamziaa @alexander-suvorov @antoine-agthe-unity