Opened 10 years ago

Closed 7 years ago

Last modified 4 years ago

#9174 closed defect (worksforme)

cygwin: robert miller's 2-descent is completely broken on cygwin

Reported by: was Owned by: tbd
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: porting: Cygwin Keywords:
Cc: jpflori, dimpase, kcrisman Merged in:
Authors: Reviewers: Karl-Dieter Crisman
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by chapoton)

sage -t  "devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx"
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1093:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    1
Got:
    0
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1098:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    2
Got:
    0
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1102:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    3
Got:
    2*log(3)/log(2) - 2
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1195:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    1
Got:
    0
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1198:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    2
Got:
    0
**********************************************************************
File "/home/wstein/sage-4.4.3/devel/sage/sage/schemes/elliptic_curves/descent_two_isogeny.pyx", line 1201:
    sage: log(n1,2) + log(n1_prime,2) - 2 # the rank
Expected:
    3
Got:
    log(3)/log(2) - 1
**********************************************************************
2 items had failures:
   3 of  30 in __main__.example_18
   3 of  11 in __main__.example_19
***Test Failed*** 6 failures.
For whitespace errors, see the file /home/wstein/.sage//tmp/.doctest_descent_two_isogeny.py
 [29.1 s]

Change History (11)

comment:1 Changed 8 years ago by kcrisman

This file passed doctests in a build of mine on XP.

comment:2 Changed 8 years ago by kcrisman

  • Milestone set to sage-duplicate/invalid/wontfix
  • Reviewers set to Karl-Dieter Crisman
  • Status changed from new to needs_review

And checking by hand works.

Since this was undoubtedly an XP machine on which the original failure was reported, I will move this to sage-invalid.

comment:3 Changed 8 years ago by kcrisman

  • Status changed from needs_review to positive_review

comment:4 Changed 8 years ago by kcrisman

  • Status changed from positive_review to needs_work

But doing anything other than the first one by hand doesn't work. In fact, nasty things happen.

comment:5 Changed 7 years ago by kcrisman

  • Cc jpflori added

I attempted this, but got forking errors; that doesn't mean it doesn't actually work. JP, want to give it a whirl?

comment:6 follow-up: Changed 7 years ago by jpflori

And the test passes for me (64bits W7 + 5.6.rc0).

comment:7 in reply to: ↑ 6 Changed 7 years ago by kcrisman

And the test passes for me (64bits W7 + 5.6.rc0).

Don't forget to try these by hand as well. In the past I've had failures only in the terminal.

comment:8 Changed 7 years ago by jpflori

  • Cc dimpase kcrisman added
  • Status changed from needs_work to needs_review

comment:9 Changed 7 years ago by jpflori

  • Status changed from needs_review to positive_review

No problems on another 32 bits W7, let's close this one.

comment:10 Changed 7 years ago by jdemeyer

  • Resolution set to worksforme
  • Status changed from positive_review to closed

comment:11 Changed 4 years ago by chapoton

  • Description modified (diff)
Note: See TracTickets for help on using tickets.