Skip to content

Fix bug in rref() function#50

Merged
Axect merged 40 commits intoAxect:devfrom
thettasch:fix-rref
Jun 14, 2022
Merged

Fix bug in rref() function#50
Axect merged 40 commits intoAxect:devfrom
thettasch:fix-rref

Conversation

@thettasch
Copy link
Copy Markdown
Contributor

Fixed an error in the function to find the reduced row echelon form of a matrix. This error causes the function to panic for certain matricies.

The pseudocode at (https://rosettacode.org/wiki/Reduced_row_echelon_form) shows that if columnCount == lead, the function should stop. The old implementation used a break statement, which only broke out of the while loop at line 3310, where it should break out of the main for loop at line 3305. This is easily fixed by using named loops to break out of a specific loop.

Axect and others added 10 commits February 10, 2022 14:21
* Add methods to control `lambda` in LM method (Axect#49)
* Weighted Uniform
* PRS
* Update `puruspe` to `0.2.0`
* Implement Cubic Hermite Spline
* Changes in `CubicSpline`
* Changes in `Calculus`
Fix bug in linspace and logspace
@Axect
Copy link
Copy Markdown
Owner

Axect commented Jun 14, 2022

@thettasch You are absolutely right! Thank you for great work! 👍

@Axect Axect changed the base branch from master to dev June 14, 2022 22:59
@Axect Axect added the bug Something isn't working label Jun 14, 2022
@Axect Axect merged commit df052bd into Axect:dev Jun 14, 2022
Axect added a commit that referenced this pull request Jun 14, 2022
* Fix bug in rref (#50)
Axect added a commit that referenced this pull request Jun 14, 2022
Fix bug in rref (#50)
@Axect
Copy link
Copy Markdown
Owner

Axect commented Jun 14, 2022

I release 0.31.4 now. It reflects your correction. 😎

@Axect Axect added good first issue Good for newcomers and removed good first issue Good for newcomers labels Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants