Opened 5 years ago

Closed 5 years ago

#21605 closed defect (wontfix)

Workaround for failing complex_double test on Cygwin

Reported by: embray Owned by:
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: porting: Cygwin Keywords:
Cc: Merged in:
Authors: Reviewers: Erik Bray
Report Upstream: N/A Work issues:
Branch: u/embray/bug/cygwin/complex_double_test (Commits, GitHub, GitLab) Commit: 77feef03b7ab487a0025a19e5b164e727d7aeb9b
Dependencies: Stopgaps:

Status badges

Description (last modified by embray)

I fixed this a while ago, but realized I don't think I ever made a ticket for it.

This test fails due to minor numerical differences (again, probably originating in the log() function) so that the polynomial returned by z.algdep(5) ends up being slightly different (it gets an extra factor of x^2). This makes the test a little more flexible to such differences.

Change History (7)

comment:1 Changed 5 years ago by embray

  • Dependencies #21292 deleted
  • Description modified (diff)
  • Milestone changed from sage-7.4 to sage-8.0
  • Status changed from new to needs_review

#21292 remains stalled, but this simple test fix doesn't require it; better to just fix this.

comment:2 Changed 5 years ago by jdemeyer

#22759 solves the same problem in a different (arguably better) way.

comment:3 Changed 5 years ago by embray

Oh, that's new--I'll have a look.

comment:4 Changed 5 years ago by embray

I see now--this should fix the underlying problem I think, but I'll have to test it.

comment:5 Changed 5 years ago by tscrim

  • Milestone changed from sage-8.0 to sage-duplicate/invalid/wontfix

Since #22759, I think we can close this as a dup.

comment:6 Changed 5 years ago by jdemeyer

  • Authors Erik Bray deleted
  • Reviewers set to Erik Bray
  • Status changed from needs_review to positive_review

comment:7 Changed 5 years ago by embray

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.