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

Add Corrector and Diagnostics attributes to MetaResult object #804

Merged
merged 5 commits into from May 23, 2023

Conversation

JulioAPeraza
Copy link
Collaborator

Closes #800 .

Changes proposed in this pull request:

  • Add Corrector and Diagnostics attributes to MetaResult object.
  • Diagnostics.transform now returns a MetaResult object.
  • Maps and tables from diagnostics are now added to maps and tables dictionaries.

@JulioAPeraza JulioAPeraza added the enhancement New feature or request label May 17, 2023
@codecov
Copy link

codecov bot commented May 17, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.04 🎉

Comparison is base (5f8ba90) 87.78% compared to head (dde34ba) 87.82%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #804      +/-   ##
==========================================
+ Coverage   87.78%   87.82%   +0.04%     
==========================================
  Files          43       43              
  Lines        5623     5642      +19     
==========================================
+ Hits         4936     4955      +19     
  Misses        687      687              
Impacted Files Coverage Δ
nimare/correct.py 95.00% <100.00%> (+0.04%) ⬆️
nimare/diagnostics.py 100.00% <100.00%> (ø)
nimare/results.py 88.60% <100.00%> (+0.44%) ⬆️
nimare/workflows/ale.py 95.06% <100.00%> (+0.18%) ⬆️
nimare/workflows/cbma.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.


contribution_table = results.tables["z_diag-Jackknife_counts"]
clusters_table = results.tables["z_clust"]
label_maps = results.maps["z_PositiveTail"]
Copy link
Member

Choose a reason for hiding this comment

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

maybe a little unintuitive what PositiveTail means in this context, may need to make it more explicit that this is a label map and not a statistical map.

Copy link
Member

@jdkent jdkent May 17, 2023

Choose a reason for hiding this comment

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

and is the label map represented as a vector or a 3D array? I think the other maps are represented as vectors.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

maybe a little unintuitive what PositiveTail means in this context, may need to make it more explicit that this is a label map and not a statistical map.

Agreed. I've added a couple of new modalities to our bids-like format. See here: https://nimare--804.org.readthedocs.build/en/804/outputs.html#file-names

and is the label map represented as a vector or a 3D array? I think the other maps are represented as vectors.

Good catch! I was missing the np.squeeze.

Thanks!

@jdkent jdkent requested a review from tsalo May 18, 2023 15:38
Copy link
Member

@jdkent jdkent 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 @JulioAPeraza!

@jdkent jdkent merged commit 476e262 into neurostuff:main May 23, 2023
19 checks passed
@JulioAPeraza JulioAPeraza deleted the enh-metaresults branch May 23, 2023 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Corrector and Diagnostics as attributes to the MetaResults object
2 participants