Skip to content

Memory optimizations to norm calculations and a fix to lazy initialization#241

Merged
IshaanDesai merged 13 commits intodevelopfrom
optimize-norm-calc
Apr 9, 2026
Merged

Memory optimizations to norm calculations and a fix to lazy initialization#241
IshaanDesai merged 13 commits intodevelopfrom
optimize-norm-calc

Conversation

@IshaanDesai
Copy link
Copy Markdown
Member

@IshaanDesai IshaanDesai commented Apr 6, 2026

This PR consists of two fixes:

  • Avoid creation of intermediate arrays in norm calculations to save memory.
  • Fix lazy initialization when there are ranks without any active simulations.

Checklist:

  • I made sure that the CI passed before I ask for a review.
  • I added a summary of the changes (compared to the last release) in the CHANGELOG.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

@IshaanDesai IshaanDesai self-assigned this Apr 6, 2026
@IshaanDesai IshaanDesai added the bug Something isn't working label Apr 6, 2026
@IshaanDesai IshaanDesai requested a review from Snapex2409 April 6, 2026 19:33
Copy link
Copy Markdown
Collaborator

@Snapex2409 Snapex2409 left a comment

Choose a reason for hiding this comment

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

I hope my comments make sense.

Comment thread micro_manager/adaptivity/adaptivity.py Outdated
Comment thread micro_manager/adaptivity/adaptivity.py Outdated
Comment thread micro_manager/micro_manager.py Outdated
Comment thread micro_manager/micro_manager.py
@IshaanDesai IshaanDesai merged commit b31027a into develop Apr 9, 2026
10 checks passed
@IshaanDesai IshaanDesai mentioned this pull request Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants