Closed lukasvo76 closed 3 years ago
Lukas,
Great - thanks for working on this! I reviewed and merged!
Tor
On Sep 1, 2021, at 4:38 AM, Lukas Van Oudenhove @.***> wrote:
@torwager https://github.com/torwager @martin-lindquist https://github.com/martin-lindquist @sgeuter https://github.com/sgeuter I made a small adaptation to fix a bug in the masking code (at least in my hands) for one scripts, and added two new scripts/functions to publish multivariate/pdm mediation results.
Please have a look and check whether you want to merge into the master branch!
You can view, comment on, or merge this pull request online at:
https://github.com/canlab/MediationToolbox/pull/21 https://github.com/canlab/MediationToolbox/pull/21 Commit Summary
small improvement to masking code created scripts/functions to publish multivariate mediation results akin to mediation_brain_results_report.m File Changes
M mediation_toolbox/mediation_brain_results_a_b_overlap.m https://github.com/canlab/MediationToolbox/pull/21/files#diff-89957de35348b866c7fcff0f239928271370b711d8946e89544db536c140ce70 (10) A mediation_toolbox/multivariate_mediation_brain_results_report.m https://github.com/canlab/MediationToolbox/pull/21/files#diff-e2ba47db847223ee6607de4ebad799c4639bd8f216e49f74ff8b005e8e658752(375) A mediation_toolbox/publish_multivariate_mediation_report.m https://github.com/canlab/MediationToolbox/pull/21/files#diff-0e9e69788b919e5be7a6a1ae0214ded2763a7caaf22949fc1bf4b34ff5875a8d (67) Patch Links:
https://github.com/canlab/MediationToolbox/pull/21.patch https://github.com/canlab/MediationToolbox/pull/21.patch https://github.com/canlab/MediationToolbox/pull/21.diff https://github.com/canlab/MediationToolbox/pull/21.diff — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/canlab/MediationToolbox/pull/21, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABPY7LGZBLJR66IAACNCQLDT7XRBZANCNFSM5DF6BW4A. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Hi Tor,
My pleasure! Working on optimizing and expanding some of the tools, especially in CANlab-help-examples too, will send pull request when I think they are ready to merge!
Thanks for the quick revision and merge!
Lukas
@torwager @martin-lindquist @sgeuter I made a small adaptation to fix a bug in the masking code (at least in my hands) for one scripts, and added two new scripts/functions to publish multivariate/pdm mediation results.
Please have a look and check whether you want to merge into the master branch!