Skip to content

Conversation

kahaaga
Copy link
Member

@kahaaga kahaaga commented Aug 4, 2024

No need for review here yet. I still have some polishing to do.

Implements #309

What is in this PR?

  • New conditional association measure ShortExpansionConditionalMutualInformation (SECMI for short).
  • New independence test SECMITest based on this measure.
  • Examples.
  • Tests.

@kahaaga kahaaga added the enhancement New feature or request label Aug 4, 2024
@codecov-commenter
Copy link

codecov-commenter commented Aug 4, 2024

Codecov Report

Attention: Patch coverage is 89.85507% with 7 lines in your changes missing coverage. Please review.

Project coverage is 94.54%. Comparing base (1054b8f) to head (c626479).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
.../short_expansion_conditional_mutual_information.jl 84.61% 4 Missing ⚠️
src/independence_tests/secmi/secmi_test.jl 91.66% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #387      +/-   ##
==========================================
- Coverage   94.65%   94.54%   -0.11%     
==========================================
  Files         111      113       +2     
  Lines        3198     3263      +65     
==========================================
+ Hits         3027     3085      +58     
- Misses        171      178       +7     

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


🚨 Try these New Features:

@kahaaga kahaaga merged commit 7bfb8b9 into main Nov 20, 2024
2 checks passed
@kahaaga kahaaga deleted the secmi branch November 20, 2024 21:17
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.

2 participants