Skip to content

Conversation

@acsenrafilho
Copy link
Member

Pull Request Template

Description

  • Solves the bug for CBFMapping memory overload

Type of Change

Please select the relevant option(s):

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📖 Documentation update (changes to documentation)
  • 🎨 Code style/formatting (non-functional changes)
  • ♻️ Code refactoring (no functional changes, no api changes)
  • ⚡ Performance improvements
  • 🧪 Test additions or improvements
  • 🔧 Build/CI configuration changes

Related Issues

Code Quality Checklist

Testing Requirements ✅

  • All existing tests pass
  • New functionality includes appropriate unit tests
  • Code follows the project's coding standards
  • Code passes linting checks (task lint-check)
  • Code includes appropriate inline documentation

Note: According to our contribution guidelines, please target the develop branch for most contributions to ensure proper testing before merging to main.

By submitting this pull request, I confirm:

  • I have read and followed the contribution guidelines
  • I have tested my changes thoroughly
  • I understand this is an open source project and my contributions may be used by others
  • I agree to the project's Code of Conduct

…ve memory estimation for processing

BUG: Fix the memory overload for the CBFMapping by not reporducing the entire m0 and pcasl volumes in the multiprocessing step (regarding get_as_numpy() copy method)
@acsenrafilho acsenrafilho self-assigned this Aug 24, 2025
@acsenrafilho acsenrafilho merged commit e2efe81 into main Aug 24, 2025
2 of 7 checks passed
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