bugGNU Octave - Bugs: bug #49410, ode solvers use Hermitian...

 
 

bug #49410: ode solvers use Hermitian conjugate where ordinary transpose is desired

Submitted by:  Rik <rik5>
Submitted on:  Fri 21 Oct 2016 10:41:57 PM UTC  
 
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: 4.2.0-rc2Operating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Sun 23 Oct 2016 10:15:19 AM UTC, comment #6:

Thanks, you are to fast for me :)

Sebastian <sschoeps>
Sun 23 Oct 2016 03:34:00 AM UTC, comment #5:

I changed the additional instances I found in comment #4 and then checked in all of the changes here (http://hg.savannah.gnu.org/hgweb/octave/rev/6b134d294d61).

Rik <rik5>
Project Administrator
Sat 22 Oct 2016 08:34:01 PM UTC, comment #4:

Seems correct. What about these additional instances in ode_event_handler?

Rik <rik5>
Project Administrator
Sat 22 Oct 2016 08:16:24 PM UTC, comment #3:

A patch for this is attached.
It also adds a test in both ode23 and ode45.
I also spotted an additional instance in
runge_kutta_inerpolate.m.

(file #38796)

Carlo de Falco <cdf>
Project Member
Sat 22 Oct 2016 07:27:05 PM UTC, comment #2:

As further confirmation consider the following example

the exact solution should be

but the actual solution has the wrong sign for the imaginary part.

Carlo de Falco <cdf>
Project Member
Sat 22 Oct 2016 06:30:42 AM UTC, comment #1:

@Rik

I think this should all be .' and not '

c.

Carlo de Falco <cdf>
Project Member
Fri 21 Oct 2016 10:41:57 PM UTC, original submission:

Attached is a list of the Hermitian conjugates used in the scripts/ode directory.

@Sebastion: If you could take a look at each instance and decide whether ' or .' is appropriate that would be great.

Rik <rik5>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #38796:  open_l3DBTYkm.txt added by cdf (4KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sschoeps (Posted a comment)
  • -unavailable- added by cdf (Posted a comment)
  • -unavailable- added by rik5 (Submitted the item)
  • -unavailable- added by rik5
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 23 Oct 2016 03:34:00 AM UTCrik5StatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Sat 22 Oct 2016 08:16:40 PM UTCcdfStatusConfirmed=>Patch Submitted
    Sat 22 Oct 2016 08:16:24 PM UTCcdfAttached File-=>Added open_l3DBTYkm.txt, #38796
    Fri 21 Oct 2016 10:41:57 PM UTCrik5Carbon-Copy-=>Added sschoeps

    Back to the top


    Powered by Savane 3.1-cleanup1