working on the rank algorithm

tries to address #5 (closed)

changelog

• the rank.rs example now only returns the rank to stdout
• two tests have been added
  • one that asserts
    \text{r}(M^T) = \text{r}(M)
  • one that asserts
    \text{r}(M) \leq \min(n, m)
    when
    n
    and
    m
    are the heigth and the width of
    M
    respectively
• Matrix::rank has been rewritten thanks to ChatGPT, the full prompt can be found in the body of 384d9559 and the algorithm has been refactored and simplified in the following few commits

Note
the two new tests do not pass on e27561bb but do pass on the tip of this MR branch, which indicates that the bugs have been solved

added 2 commits

• c02acb50 - ask ChatGPT for the correct rank algorithm
• adae627e - fix the algorithm in two places

Compare with previous version

added 2 commits

• 384d9559 - ask ChatGPT for the correct rank algorithm
• 553c4f78 - fix the algorithm in two places

Compare with previous version

added 4 commits

• c4b308f3 - simplify the last part of the computation
• f9897e55 - use Matrix wrappers
• e0414de5 - use `T::is_zero`
• 5245cdda - refactor into `Matrix::swap_rows`

Compare with previous version

marked this merge request as ready

requested review from @j.detchart

assigned to @a.stevan

changed the description

added 1 commit

• 912e504a - add two comments to explain parts of the algorithm

Compare with previous version

mentioned in merge request !38 (closed)

approved this merge request

added 2 commits

• 9fe62d5a - 1 commit from branch main
• 04cae335 - Merge remote-tracking branch 'origin/main' into rank

Compare with previous version

enabled an automatic merge when the pipeline for 04cae335 succeeds

merged

mentioned in commit 1171bc3e