bugGNU Octave - Bugs: bug #48285, FAIL's in mapper.cc-tst

 
 

bug #48285: FAIL's in mapper.cc-tst

Submitter:  None
Submitted:  Tue 21 Jun 2016 11:21:48 AM UTC
   
 
Category:  Libraries Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Inaccurate Result
Status:  Fixed Assigned to:  None
Originator Name:  Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * GNU/Linux Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 30 Jun 2016 09:18:33 PM UTC, comment #5: 

I have turned the four failing tests into xtests to mark them as expected failures, since there appear to be a lot of systems out there with math libraries that are broken:

http://hg.savannah.gnu.org/hgweb/octave/rev/18af3ea1d387

This does not address the two failing tests that appear to be off by some small tolerance, this only addresses the four tests explicitly added to make sure that the functions return the right asymptotic values for large magnitude arguments (i.e., bug #45507), which the broken math libraries don't.

If someone really wants to fix the other (not newly added) tests that are now failing, also due to older buggy math libraries, that can be reported and handled separately.

Mike Miller <mtmiller>
Group Member
Tue 21 Jun 2016 08:22:02 PM UTC, comment #4: 

configure scripts detects these functions exist.
From config.status:


D["HAVE_COMPLEX_STD_ACOS"]=" 1"
D["HAVE_COMPLEX_STD_ACOSH"]=" 1"
D["HAVE_COMPLEX_STD_ASIN"]=" 1"
D["HAVE_COMPLEX_STD_ASINH"]=" 1"
D["HAVE_COMPLEX_STD_ATAN"]=" 1"
D["HAVE_COMPLEX_STD_ATANH"]=" 1"


See also attached config.log part config_log.txt

(file #37537)

Anonymous
Tue 21 Jun 2016 03:27:31 PM UTC, comment #3: 

Yes, the failing tests look like the ones I added recently to ensure correct output, based empirically on what I see with GCC 5.4 on x86_64 after the fix for bug #45507. These are new tests and it's good that they are failing so you can help us to identify what additional work is needed, if any.

a.rburgers, what does configure show on your system for "checking for std::acos in <complex>" and the following lines? The GCC 4.9 release notes claim to support these functions.

Mike Miller <mtmiller>
Group Member
Tue 21 Jun 2016 01:58:29 PM UTC, comment #2: 

It might be that the C++ math library is bad, or that it is not present and you are using the fallback routines within Octave.

It is probably related to this cset:


changeset:   21945:e9765b62d4e8
user:        Mike Miller <mtmiller@octave.org>
date:        Fri Jun 17 14:24:27 2016 -0700
summary:     Use C++11 standard complex trig functions when available (bug #44310, bug #45507)


See bug #45507 in particular where it was reported that gcc 4.8.2 did not have correct implementations for some of the trig functions.

Rik <rik5>
Group administrator
Tue 21 Jun 2016 11:38:39 AM UTC, comment #1: 

on ubuntu 16.04 I get
Summary:

  PASS     14034
  FAIL         0
  XFAIL       29
  SKIPPED     65

for the same ID number.

Doug Stewart <dastew>
Tue 21 Jun 2016 11:21:48 AM UTC, original submission:  

for HG-ID 6da01de4dd8a there are these 6 FAIL's.
These failures have started appearing quite recently.
See attached for mapper.cc part of fntest.log


libinterp/corefcn/mappers.cc-tst ..........
PASS 390/396
FAIL    6


This is on RHEL 6.8, gcc 4.9.3

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #37537:  config_log.txt added by None (1KiB - text/plain)
file #37536:  mapper_fntest_log.txt added by None (3KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by rik5
  • -email is unavailable- added by dastew (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-06-30 mtmiller StatusNeed Info Fixed
        Open/ClosedOpen Closed
    2016-06-21 None Attached File- Added config_log.txt, #37537
    2016-06-21 mtmiller CategoryInterpreter Libraries
        Item GroupRegression Inaccurate Result
        StatusNone Need Info
    2016-06-21 rik5 Carbon-Copy- Added mtmiller
    2016-06-21 None Attached File- Added mapper_fntest_log.txt, #37536

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code