Matrix.prototype.determinant の追加

• Reporter: revulo
• Owner: dhrname
• Type: パッチ報告
• Status: Closed
• Component: (None)
• MileStone: (None)
• Priority: 1 - Lowest
• Severity: 5 - Medium
• Resolution: Fixed
• File: 1

Details Reply

Matrix オブジェクトに、行列式を求めるメソッドを追加して
みました。この方が分かりやすくなると思うのですが、どう
でしょうか？

Ticket History (3/6 Histories)

2008-08-24 22:18 Updated by: revulo

• File 3307: sie_determinant.patch is attached

2008-08-25 20:25 Updated by: dhrname

• Resolution Update from None to Accepted
• Owner Update from (None) to dhrname

Comment Reply

Logged In: YES
user_id=32883

確かにわかりやすいです。
さっそく、使わせていただきます。ありがとうございます。

2008-08-28 01:03 Updated by: revulo

Comment Reply

Logged In: YES
user_id=33975

後で思ったのですが、determinant の本当の値は ad-bc なのに絶
対値を返すのは、メソッドの名前と内容が一致していないです。呼
び出す側で Math.sqrt(Math.abs(matrix.determinant())) のよう
にする方がいいのかもしれません。

ただそれだと、なんとなく分かりにくい気がするので、他に行列式
の値を使わないのであれば、そのままでもいいのかなと思いまし
た。

2008-08-28 20:14 Updated by: dhrname

Comment Reply

Logged In: YES
user_id=32883

言葉の意味を考慮して、Math.sqrt(Math.abs(matrix.determinant
())) という形式をリビジョン448で採用しました。

2008-09-15 18:43 Updated by: dhrname

• Priority Update from 5 - Medium to 1 - Lowest
• Resolution Update from Accepted to Fixed

2008-09-16 20:53 Updated by: dhrname

• Status Update from Open to Closed
• Ticket Close date is changed to 2008-09-16 20:53