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 Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  None Assigned to:  None
Originator Name:  Open/Closed:  Open
Release:  other Operating 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 Files:

No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by hardy (Posted a comment)
  • -email is unavailable- added by rlaboiss (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 project members can vote.




    No changes have been made to this item

    Back to the top

    Powered by Savane 3.4