Feature #10057

[PATCH] Add tests for Matrix class.

Added by gogo tanaka about 1 year ago. Updated about 1 year ago.

[ruby-core:63809]
Status:Closed
Priority:Normal
Assignee:-

Description

I send 7 patches.

In one of patches (reorder_Matrix#real_and_Matrix_imaginary.patch), I just reorder methods.

I think Matrix#real should be upon Matrix#imaginary(From what I see complex.c)

In the others(6 patches), I add tests for Matrix class.

add_test_for_Matrix#eigensystem.patch Magnifier (594 Bytes) gogo tanaka, 07/18/2014 05:47 AM

add_test_for_Matrix#lup.patch Magnifier (644 Bytes) gogo tanaka, 07/18/2014 05:47 AM

add_test_for_Matrix#conjugate.patch Magnifier (775 Bytes) gogo tanaka, 07/18/2014 05:47 AM

add_test_for_Matrix#real.patch Magnifier (504 Bytes) gogo tanaka, 07/18/2014 05:47 AM

add_test_for_Matrix#imaginary.patch Magnifier (485 Bytes) gogo tanaka, 07/18/2014 05:47 AM

add_test_for_Matrix#rect.patch Magnifier (512 Bytes) gogo tanaka, 07/18/2014 05:47 AM

Associated revisions

Revision 46861
Added by Hiroshi SHIBATA about 1 year ago

  • test/matrix/test_matrix.rb: Add tests for Matrix class. [Feature #10057]

Revision 46861
Added by Hiroshi SHIBATA about 1 year ago

  • test/matrix/test_matrix.rb: Add tests for Matrix class. [Feature #10057]

History

#1 Updated by Hiroshi SHIBATA about 1 year ago

Please split new feature and tests into another issue.

#2 Updated by gogo tanaka about 1 year ago

@ Hiroshi SHIBATA さん

I split patch "reorder_Matrix#real_and_Matrix_imaginary.patch" to https://bugs.ruby-lang.org/issues/10058

How can I delete patch from feature ?

承知致しました.

7つのPATCHの内、"reorder_Matrix#real_and_Matrix_imaginary.patch"

はこちら https://bugs.ruby-lang.org/issues/10058 に切り出させて頂きました.

一度featureに添付したPATCHは削除出来ないのでしょうか.

#3 Updated by Hiroshi SHIBATA about 1 year ago

  • File deleted (reorder_Matrix#real_and_Matrix_imaginary.patch)

#4 Updated by Hiroshi SHIBATA about 1 year ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r46861.


  • test/matrix/test_matrix.rb: Add tests for Matrix class. [Feature #10057]

#5 Updated by Marc-Andre Lafortune about 1 year ago

Gogo Tanaka san,

Thanks for these additional tests.

Please note that the vast majority of tests of the Matrix library are in RubySpec: https://github.com/rubysl/rubysl-matrix

I hope you'll find them extensive, having for instance many more tests than found in this patch imaginary. If you'd like to port them this would be great.

#6 Updated by gogo tanaka about 1 year ago

@ Hiroshi SHIBATA さん

ファイルの削除や進捗、ステータスのハンドリングありがとうございます.

Thank you for your some supports.

@ Mr. Marc-Andre Lafortune

I don't know that ... ! it's great! Thank you for your kindness!!!

Also available in: Atom PDF