Project

General

Profile

Actions

Bug #6290

closed

Matrix#hermitian? fails to check diagonal elements

Added by kernigh (George Koehler) about 12 years ago. Updated about 12 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 2.0.0dev (2012-04-14 trunk 35320) [x86_64-openbsd5.0]
Backport:
[ruby-core:44344]

Description

=begin
A Hermitian matrix is equal to its own conjugate transpose.

matrix == matrix.conj.t

The bug is that Matrix#hermitian? fails to check the diagonal elements of the matrix. For example, the matrix in the next script is ((not)) Hermitian:

require 'matrix'
i = Complex::I
p Matrix[[-2, -3-i, 3+i ],
[-3+i, 3, -5-7i],
[ 3-i, -5+7
i, 9-4*i]].hermitian?

This script prints (({true})). I expected (({false})), because 9-4i != 9+4i. Each diagonal element must equal its own conjugate (so, its imaginary part must be zero).

Matrix#symmetric? can skip the diagonal elements, because each number equals itself. Matrix#hermitian? must not skip the diagonal elements, because some number might not equal its own conjugate.

I attach a git diff with a simple fix to Matrix#hermitian?, and some test.
=end


Files

hermitian-fix.diff (1.24 KB) hermitian-fix.diff a simple fix to Matrix#hermitian?, and some test kernigh (George Koehler), 04/14/2012 03:48 AM

Updated by marcandre (Marc-Andre Lafortune) about 12 years ago

  • Category set to lib
  • Status changed from Open to Assigned
  • Assignee set to marcandre (Marc-Andre Lafortune)
  • Target version set to 2.0.0
Actions #2

Updated by marcandre (Marc-Andre Lafortune) about 12 years ago

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

This issue was solved with changeset r35344.
George, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/matrix.rb (hermitian?): Bug fix, patch by George Koehler
    [Bug #6290] [rubyspec:4b9573d7613]
Actions

Also available in: Atom PDF

Like0
Like0Like0