patchGNU Octave - Patches: patch #8172, Improve complex mapper functions...


patch #8172: Improve complex mapper functions (see Numerical, item 2 on Projects page)

Submitted by:  Craig Hudson <craighudson>
Submitted on:  Fri 30 Aug 2013 01:04:39 PM UTC  
Category:  Core : new feature Priority:  3 - Low
Status:  Wont Do Privacy:  Public
Assigned to:  None Open/Closed:  Closed

Add a New Comment (Rich Markup)

You are not logged in

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


Sat 18 Jan 2020 12:29:50 PM UTC, comment #2: 

I acknowledged your work in the wiki

Kai Torben Ohlhus <siko1056>
Project Member
Sat 18 Jan 2020 12:25:08 PM UTC, comment #1: 

Review for Octave 6.

No interest for 6 years.  Very sorry, but this patch with >1000 lines of code does not apply to the Octave development branch any longer.

If you face problems with complex numbers.  Please file a bug report about smaller chunks of code to work on.

Closing item.

Kai Torben Ohlhus <siko1056>
Project Member
Fri 30 Aug 2013 01:04:39 PM UTC, original submission:  

This includes code for making use of signed zero, as described in Branch Cuts for Complex Elementary Functions, or Much Ado About Nothing's Sign Bit by W. Kahan. Note that casts of complex numbers with imaginary part equal to zero to real numbers previously occurred with both positive and negative zero, but now only occurs with positive zero. There is a case for not doing this narrowing conversion with either sign of zero, as (eg) dividing a real number by a complex number with imaginary part equal to positive zero gives the wrong sign of the zero imaginary part to the result. eg 1/(1 + 0*i) should be 1 - 0*i, but is calculated as 1 due to the denominator being cast to a real value.

Craig Hudson <craighudson>


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

Attach Files:

Attached Files
file #28959:  change.diff added by craighudson (46KiB - text/x-patch)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by craighudson (Submitted the item)

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

    Only logged-in users can vote.




    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-01-18 siko1056 CategoryNone => Core : new feature
        Priority5 - Normal => 3 - Low
        StatusNone => Wont Do
        Open/ClosedOpen => Closed
    2013-08-30 craighudson Attached File- => Added change.diff, #28959

    Back to the top

    Powered by Savane 3.7