la4j icon indicating copy to clipboard operation
la4j copied to clipboard

New vector methods: NormalizeInPlace

Open vkostyukov opened this issue 10 years ago • 5 comments

Vector {
  Vector normalize(VectorAccumulator acc);
  void normalizeInPlace(VectorAccumulator acc);
}

vkostyukov avatar May 22 '14 04:05 vkostyukov

Does 'normalize' mean construction of a vector with length equals 1 or something else? I'm not fully understand why do we need VectorAccumulator as method argument.

SamoylovMD avatar Aug 04 '14 11:08 SamoylovMD

Normalize means divide the vector by it's norm. We need accumulator in order to pass a norm-function.

vkostyukov avatar Aug 04 '14 12:08 vkostyukov

Please, note that we don't need in-place version for upcoming release.

vkostyukov avatar Aug 04 '14 12:08 vkostyukov

Just leaving a note here that:

Vector normalize()
Vector normalize(VectorAccumulator acc)

were both added in #206 however:

void normalizeInPlace(VectorAccumulator acc)

was not.

DWiechert avatar Dec 03 '14 03:12 DWiechert

Moving the rest of this task to the next milestone.

vkostyukov avatar Dec 03 '14 23:12 vkostyukov