Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add isEchelonForm and isReducedEchelonForm #84

Merged
merged 1 commit into from
Feb 9, 2019

Conversation

Goneiross
Copy link
Contributor

No description provided.

@Goneiross Goneiross changed the title Add isEchelonForm and isReducedEchelonForm asked in issues Add isEchelonForm and isReducedEchelonForm Feb 3, 2019
@Goneiross Goneiross force-pushed the isEchelonForm branch 2 times, most recently from 0ec0b76 to 815cb92 Compare February 9, 2019 10:27
@codecov
Copy link

codecov bot commented Feb 9, 2019

Codecov Report

Merging #84 into master will increase coverage by 0.61%.
The diff coverage is 95.12%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   61.32%   61.94%   +0.61%     
==========================================
  Files          25       25              
  Lines        2211     2252      +41     
  Branches      271      280       +9     
==========================================
+ Hits         1356     1395      +39     
- Misses        777      779       +2     
  Partials       78       78
Impacted Files Coverage Δ
src/abstractMatrix.js 63.66% <95.12%> (+1.62%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b48178...6a02897. Read the comment docs.

@lpatiny lpatiny requested a review from targos February 9, 2019 10:48
@lpatiny
Copy link
Member

lpatiny commented Feb 9, 2019

@targos It is an important property of a matrix. Is this ok to merge in the project ?

@targos targos merged commit dee2a94 into mljs:master Feb 9, 2019
@targos
Copy link
Member

targos commented Feb 9, 2019

Thank you @Goneiross !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants