<div dir="ltr">

<p class="">Hi Alex,</p>

<p class="">Thanks for looking into this function.<span style>  </span>You’re right that ft_freqcomparison
initializes the output structure ‘ freq’ (line 87) using the first input structure.
This could indeed be problematic when the two input structures are dissimilarly
sized. <span style> </span>In fact, having a size difference
in the powspctrm fields between the two input structure is an exclusion criterion
for using this function in the first place, e.g. because it may result in
comparing frequency powers of different channels, trials, or subjects. For that
reason, a check was built in at lines 92 and 115, which ensures that the input
structures have identical matrix size. Although not intuitive clear maybe from
a reader’s perspective, this is thus not an issue and it will not affect your
data.</p>

<p class="">Also please note that ft_freqcomparison has become outdated with
the arrival of ft_math. Technically, this function can do the same, but with
more flexibility and better bookkeeping. <span style> </span></p>

<p class="">Best wishes,</p>

<p class="">Arjen</p>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/3/13 Alexander Maye <span dir="ltr"><<a href="mailto:a.maye@uke.de" target="_blank">a.maye@uke.de</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi All,<br>
<br>
I just spotted something that could be a bug in ft_freqcomparison.m<br>
(fieldtrip-20130312 and previous versions). Don't now if this is the place for<br>
report, but if this really is an issue everyone using this function may be<br>
interested.<br>
<br>
The problem is in line 87 where freq = varargin{1}; is initialized to the<br>
*first* freq argument, that according to the doc is used as the norm. So the<br>
output (powspctrm) has the dimension of the norm, not the second freq<br>
argument, that one wants to become normalized. Strange, hmm? Then, in the<br>
following for-loops the index variable runs over the size of the *second* freq<br>
argument, updating trials in the powspctrm with the size of the *first* freq<br>
argument, which seems simply wrong.<br>
<br>
Maybe this is an ingenious programming trick, but initializing freq =<br>
varargin{2}; on line 87 would make more sense from my point of view.<br>
<br>
Best,<br>
<br>
                                 ALEX.<br>
<br>
<br><p><br>
--<br>
Pflichtangaben gemäß Gesetz über elektronische Handelsregister und Genossenschaftsregister sowie das Unternehmensregister (EHUG):</p>

<p>Universitätsklinikum Hamburg-Eppendorf; Körperschaft des öffentlichen Rechts; Gerichtsstand: Hamburg</p>

<p>Vorstandsmitglieder: Prof. Dr. Martin Zeitz (Vorsitzender), Dr. Alexander Kirstein, Joachim Prölß, Prof. Dr. Dr. Uwe Koch-Gromus </p><br>_______________________________________________<br>
fieldtrip mailing list<br>
<a href="mailto:fieldtrip@donders.ru.nl">fieldtrip@donders.ru.nl</a><br>
<a href="http://mailman.science.ru.nl/mailman/listinfo/fieldtrip" target="_blank">http://mailman.science.ru.nl/mailman/listinfo/fieldtrip</a><br></blockquote></div><br></div>