QR Decomposition Method Object Refactoring#288

Merged

hemalvarambhia merged 33 commits intomasterfrom

qr-method-object-refactoring

Sep 5, 2022

Commits on Aug 28, 2022

Added an example discovered in wikipedia that we can use to get a deeper understanding.
hemalvarambhia
committed
Added a test to see how QR Decomposition works and then began migrating logic to the new method object.
hemalvarambhia
committed
Clarified the classification of a method.
hemalvarambhia
committed
Copied the logic to a more sensible place.
hemalvarambhia
committed
Replaced logic with a method that does the same thing.
hemalvarambhia
committed
Made the code more intention-revealing.
hemalvarambhia
committed
Extracted a computation to an intention revealing variable.
hemalvarambhia
committed
Broke up the computation of the indices. The computation i appears to be a matrix of minor.
hemalvarambhia
committed
Extracted a computation to a more sensible place in an intention-revealing method.
hemalvarambhia
committed
Added a test that computes the minor of a square matrix.
hemalvarambhia
committed
Added another assertion to make logic of method clear.
hemalvarambhia
committed
Added another assertion to raise confidence.
hemalvarambhia
committed
Another another interesting test where we compute the minor of a non-square matrix.
hemalvarambhia
committed
Improved code formatting, replaced duplicate code with a message send.
hemalvarambhia
committed

Commits on Aug 30, 2022

Inlined some local variables, made the decomposeWithPivot method look more similar to decompose.
hemalvarambhia
committed

Commits on Aug 31, 2022

Used a more sensible name, derived from the wikipedia entry.
hemalvarambhia
committed

Commits on Sep 1, 2022

Improved the name of the temporary variable.
hemalvarambhia
committed

Commits on Sep 4, 2022