Skip to content

Changes from background agent bc-11074d04-42c8-4e64-828a-811613b5e9e3#4

Draft
buildwithsuhana wants to merge 1 commit intofix_optmfrom
cursor/bc-11074d04-42c8-4e64-828a-811613b5e9e3-4b93
Draft

Changes from background agent bc-11074d04-42c8-4e64-828a-811613b5e9e3#4
buildwithsuhana wants to merge 1 commit intofix_optmfrom
cursor/bc-11074d04-42c8-4e64-828a-811613b5e9e3-4b93

Conversation

@buildwithsuhana
Copy link
Owner

Fixes incorrect memory usage reporting in CoordinatedOptimizer to show accurate total, sharded, and saved memory.

Previously, memory usage was incorrectly reported as 0.00 MB or 40 bytes. This PR updates the CoordinatedOptimizer to correctly identify optimizer states, use zero buffers for accurate size estimation during sharding, and compute memory usage and savings based on variable shapes and dtypes.


Open in Cursor Open in Web

Co-authored-by: suhanagun <suhanagun@gmail.com>
@cursor
Copy link

cursor bot commented Aug 25, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

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.

2 participants