Skip to content
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.

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

TEST: Check and filter PCA dimensionality problem warning #3004

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Dec 10, 2023

Check and filter PCA dimensionality problem warning in MPPCA test.

Fixes:

denoise/tests/test_lpca.py::test_mppca_in_phantom
 denoise/localpca.py:295: UserWarning:
 Number of samples 27 - 1 < Dimensionality 104.
 This might have a performance impact.
 Increase patch_radius to 2 to avoid this.
    warn(dimensionality_problem_message(arr, num_samples, spr), UserWarning)

raised, for example, at:
https://github.com/dipy/dipy/actions/runs/7159954273/job/19493785864#step:9:3932

Left behind in commit fe33ea9.

@jhlegarreta
Copy link
Contributor Author

Cross-referencing PR #3001.

@jhlegarreta
Copy link
Contributor Author

The warning is gone.

Copy link

codecov bot commented Dec 10, 2023

Codecov Report

Merging #3004 (52a069f) into master (fa8cb22) will decrease coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3004      +/-   ##
==========================================
- Coverage   81.78%   81.76%   -0.02%     
==========================================
  Files         146      146              
  Lines       20566    20566              
  Branches     3281     3281              
==========================================
- Hits        16820    16816       -4     
- Misses       2911     2914       +3     
- Partials      835      836       +1     

see 1 file with indirect coverage changes

@jhlegarreta jhlegarreta changed the title TEST: Check for and filter PCA dimensionality problem warning TEST: Check and filter PCA dimensionality problem warning Dec 11, 2023
Check and filter PCA dimensionality problem warning in MPPCA test.

Fixes:
```
denoise/tests/test_lpca.py::test_mppca_in_phantom
 denoise/localpca.py:295: UserWarning:
 Number of samples 27 - 1 < Dimensionality 104.
 This might have a performance impact.
 Increase patch_radius to 2 to avoid this.
    warn(dimensionality_problem_message(arr, num_samples, spr), UserWarning)
```

raised, for example, at:
https://github.com/dipy/dipy/actions/runs/7159954273/job/19493785864#step:9:3932

Left behind in commit fe33ea9.
Copy link
Member

@skoudoro skoudoro left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @jhlegarreta !

@skoudoro skoudoro merged commit 50e8ca9 into dipy:master Dec 11, 2023
31 checks passed
@jhlegarreta jhlegarreta deleted the FixPCAWarnings2 branch December 11, 2023 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants