Skip to content

Conversation

@acsenrafilho
Copy link
Member

Pull Request Template

Description

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 added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Aug 24, 2025
@codecov
Copy link

codecov bot commented Aug 24, 2025

Codecov Report

❌ Patch coverage is 83.87097% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
asltk/reconstruction/cbf_mapping.py 82.60% 4 Missing ⚠️
asltk/aux_methods.py 83.33% 1 Missing ⚠️
Files with missing lines Coverage Δ
asltk/reconstruction/ultralong_te_mapping.py 99.08% <100.00%> (+0.01%) ⬆️
asltk/aux_methods.py 89.70% <83.33%> (+0.99%) ⬆️
asltk/reconstruction/cbf_mapping.py 95.65% <82.60%> (-4.35%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@acsenrafilho acsenrafilho merged commit 257462c into develop Aug 24, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants