Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[FIX] concordant z values #125

Merged
merged 4 commits into from
Sep 11, 2024
Merged

Conversation

jdkent
Copy link
Member

@jdkent jdkent commented Sep 11, 2024

only positive z-values were displayed even if the p-value came from the "less than" array, this fixes that error.

Copy link

codecov bot commented Sep 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.09%. Comparing base (624968e) to head (046d36c).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
+ Coverage   91.06%   91.09%   +0.02%     
==========================================
  Files          13       13              
  Lines         918      932      +14     
==========================================
+ Hits          836      849      +13     
- Misses         82       83       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jdkent jdkent merged commit 39f579d into neurostuff:master Sep 11, 2024
13 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant