Free cookie consent management tool by TermsFeed Policy Generator

Changes between Initial Version and Version 1 of Ticket #1913, comment 24


Ignore:
Timestamp:
12/05/12 15:20:18 (11 years ago)
Author:
gkronber
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #1913, comment 24

    initial v1  
    11Further comments from reviewing the NcaInitializers:
    22 * In `LdaInitializer` line 47: this is effectively equivalent to PrepareInputMatrix()... use the existing method instead (also in the other PcaInitializer).
    3  * `RandomInitializer` line 76:
    4    `matrix[i] = random.NextDouble() / range[i / dimensions];`
    5    isn't `i / dimensions` always zero since both are ints?
     3~~ * `RandomInitializer` line 76:   `matrix[i] = random.NextDouble() / range[i / dimensions];`   isn't `i / dimensions` always zero since both are ints?~~
    64