<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">
Dear Sean,
<div class=""><br class="">
</div>
<div class="">Thanks for catching this. The original line 852 does not make sense at all, indeed.</div>
<div class="">May I suggest to replace this with data.fourierspctrm(:,:,i).’ (note the ‘.’)?</div>
<div class="">Can you please submit this as a pull request on github? <a href="http://www.fieldtriptoolbox.org/development/git" class="">http://www.fieldtriptoolbox.org/development/git</a></div>
<div class=""><br class="">
</div>
<div class=""><br class="">
</div>
<div class="">Many thanks,</div>
<div class=""><br class="">
</div>
<div class="">Jan-Mathijs</div>
<div class=""><br class="">
</div>
<div class=""><br class="">
</div>
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">On 14 Nov 2018, at 21:26, SEAN TANABE <<a href="mailto:stanabe@wisc.edu" class="">stanabe@wisc.edu</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div id="divtagdefaultwrapper" dir="ltr" style="font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; font-size: 12pt; font-family: Calibri, Helvetica, sans-serif;" class="">
<div style="margin-top: 0px; margin-bottom: 0px;" class=""><span style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">Dear all, </span></div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
I am trying to use the powcorr_orth method in ft_connectivityanalysis and noticed the results always produce a random connectivity pattern. A similar issue has been raised a year ago here; </div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<a href="https://mailman.science.ru.nl/pipermail/fieldtrip/2017-July/011695.html" class="">https://mailman.science.ru.nl/pipermail/fieldtrip/2017-July/011695.html</a><br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
I found after editing a line from ft_connectivityanalysis the results look more reasonable, and wanted to confirm if this editing is correct with the fieldtrip developers. </div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
In fieldtrip-20170820 ft_connectivityanalysis I noticed a line which reshapes the fourierspctrm before inputting to ft_connectivity_powcorr_orth (<span style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">line 852 in fieldtrip-20181113</span>). </div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
dat = reshape(data.fourierspctrm(:,:,i)', nrpttap, nchan).';<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
datout{i} = ft_connectivity_powcorr_ortho(dat, optarg{:});</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
This seems to not fit the description of ft_connectivity_powcorr_orth, which requires "dat" to have nchan rows. </div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
I changed this to</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<div class=""> dat = data.fourierspctrm(:,:,i)';</div>
<div class=""> datout{i} = ft_connectivity_powcorr_ortho(dat, optarg{:});</div>
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
The above gave natural looking blobs of connectivity in the topology. </div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
Thank you.</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
Sean</div>
<div style="color: rgb(34, 34, 34); font-family: arial, sans-serif; font-size: small;" class="">
<br class="">
</div>
<br class="">
<p style="margin-top: 0px; margin-bottom: 0px;" class=""></p>
</div>
<span style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">_______________________________________________</span><br style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">fieldtrip
mailing list</span><br style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<a href="https://mailman.science.ru.nl/mailman/listinfo/fieldtrip" style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">https://mailman.science.ru.nl/mailman/listinfo/fieldtrip</a><br style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">
<a href="https://doi.org/10.1371/journal.pcbi.1002202" style="font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">https://doi.org/10.1371/journal.pcbi.1002202</a></div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>