Skip to content

[NET-225] Result P-values#75

Merged
pollaro merged 3 commits intodevelopmentfrom
NET225_resultfields
Nov 4, 2024
Merged

[NET-225] Result P-values#75
pollaro merged 3 commits intodevelopmentfrom
NET225_resultfields

Conversation

@pollaro
Copy link
Copy Markdown
Collaborator

@pollaro pollaro commented Nov 2, 2024

There's a lot here. Let's start with what is desired:

  • No Permutations
    • Two-sample p-value for chi-squared and hypergeo
    • Single-sample p-value for everything else
    • legacy p-value
  • Full Connectome
    • Two sample p-value for everything
    • legacy p-value
    • winkler p-value
    • westfall-young p-value
  • Within Network Pair
    • Same as no permutations
    • winkler p-value
    • westfall-young p-value

Added:

  • static method to NetworkTestResult to aid in determining single-sample or two-sample
  • lots of places adding single_sample_ or two_sample_
  • changes to all the network tests to account for new names

create only needed results in each method
delete two sample no-permutations after ranking
function to get p-value names
renaming in tests and plotting.
@pollaro pollaro changed the title rename p-values to be more consistent. [NET-225] Result P-values Nov 4, 2024
@pollaro pollaro merged commit 56e2307 into development Nov 4, 2024
@pollaro pollaro deleted the NET225_resultfields branch November 4, 2024 15:52
pollaro added a commit that referenced this pull request Mar 25, 2025
* rename p-values to be more consistent.
create only needed results in each method

* changed names to better names, created static method

* lots of changes.
delete two sample no-permutations after ranking
function to get p-value names
renaming in tests and plotting.
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.

1 participant