You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Allow LocalPermutationTest to be used with TEShannon estimated using dedicated TE estimators (#350)
* Fix issue #348
* More effective estimation for `Lindner` when doing e.g. surrogate tests
partially addresses #344
* Typos
* Add tests
* Up patch version
* Correctly scale
* Make sure we have enough samples for tests
* Better test organization
* It is the estimator that controls what happens, not the measure
* Add note to `LocalPermutationTest` docstring about transfer entropy
* Error should occur only for `TransferEntropyEstimator`s
* More tests
* Improve test comments.
* Fix#349
And also mention conditioning in `Zhu1` docs
Copy file name to clipboardExpand all lines: changelog.md
+2
Original file line number
Diff line number
Diff line change
@@ -5,6 +5,8 @@
5
5
### Bug fixes
6
6
7
7
- Fixed bug in `transferentropy` function which yielded identical results in both directions for the bivariate case.
8
+
- Fixed bug that occurred when using `LocalPermutationTest` with `TEShannon` as the measure and a dedicated `TransferEntropyEstimator` (e.g. `Zhu1` or `Lindner`). This occurred because the `LocalPermutationTest` is, strictly speaking, a test using conditional mutual information as the measure. Therefore, naively applying a `TransferEntropy` measure such as `TEShannon` would error. This is fixed by performing a similar procedure where the source marginal is shuffled according to local neighborhoods in the conditional marginal. This is similar, but not identical to the CMI-based `LocalPermutationTest`, and adapts to the specific case of transfer entropy estimation using dedicated transfer entropy estimators instead of some lower-level estimator.
9
+
- Fixed bug in `Zhu1` transfer entropy estimator where when box volumes were extremely small, taking the logarithm of volume ratios resulted in `Inf` values. This was solved by simply ignoring these volumes.
| [`TEShannon`](@ref) | ✓ | ✓ | Yes | Pairwise tests not possible with `TransferEntropyEstimator`s, only lower-level estimators, e.g. `FPVP`, `GaussianMI` or `Kraskov` |
81
+
| [`PMI`](@ref) | ✖ | ✓ | Yes | |
82
82
83
83
The `LocalPermutationTest` is only defined for conditional independence testing.
84
84
Exceptions are for measures like [`TEShannon`](@ref), which use conditional
0 commit comments