bugGNU Octave - Bugs: bug #51980, [octave forge] (image) Failing...

 
 

bug #51980: [octave forge] (image) Failing unit tests for edge

Submitted by:  Rafael Laboissiere <rlaboiss>
Submitted on:  Sun 10 Sep 2017 03:06:41 PM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: NoneAssigned to: None
Originator Name: Open/Closed: Open
Release: otherOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Mon 20 Nov 2017 07:37:51 PM UTC, comment #3:

> If tests are expected to fail, then they should be in "%!xtest" sections instead of "%!test" sections.


This depends on how one interprets "expected to fail". The ones
failing are not expected to fail, they are bugs that we are real bugs
that we are aware of. They should still be fixed, it just happens
that we haven't got around fixing those issues either because we do
not know what Matlab is actually doing to achieve those results or
because no one has found the time to work on them.

My feeling is that marking tests for known bugs as expected to fail
helps in having them never fixed but it also seems to be causing a lot
of confusion. It makes really hard to use the test suite to identify
regressions which reduces a lot the usefulness of tests so I will be
marking this as xtest.

Carnë Draug <carandraug>
Project Member
Sun 10 Sep 2017 08:59:16 PM UTC, comment #2:

Thanks for the comments, Hartmut.

If tests are expected to fail, then they should be in "%!xtest" sections instead of "%!test" sections.

Rafael Laboissiere <rlaboiss>
Sun 10 Sep 2017 06:04:10 PM UTC, comment #1:

I think those two failing tests in edge.m (for the automatic threshold value in the "Canny" method) are intentional.

When I remember Carne right, they should document the remaining Matlab-incompatibility of this Octave function. See bug #46099 for details.

(Note: I think I heard Carne say that intentionally failing tests in the image package are not so uncommon...)

Hartmut <hardy>
Sun 10 Sep 2017 03:06:41 PM UTC, original submission:

The two last unit tests for function edge in the image package are failing. The test log is below.

Rafael Laboissiere <rlaboiss>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by hardy (Posted a comment)
  • -unavailable- added by rlaboiss (Submitted the item)
  •  

    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):

     

     

    No Changes Have Been Made to This Item

    Back to the top


    Powered by Savane 3.1-cleanup1