Closed pietromarchesi closed 6 years ago
This is a nice idea. It's fairly straightforward to do, I'll just add a new option to the dictionary settings for each MI/CMI/TE calculator to specify the algorithm and instantiate the right calculator. Nothing else would need changing.
Ok - Patricia I have code for this ready. Shall I push it along with the other changes we discussed recently re memory exhaustion or separately?
Sounds good, thanks @pietromarchesi for bringing this up, and thanks @jlizier for the quick reply and implementation!
Apologies for the delay in sending the PR through here, got caught up on other things as usual ...
Merged PR #18 in commit 31d96672e2e359cad75092e18da6950fff43555f.
To be able to use both Kraskov algorithms, I have written my own wrappers to the JIDT calculators, but perhaps it would be nice to be able to access both from IDTxl too? I was thinking of extending the IDTxl class for my own use, but if it's useful I could submit a PR.