Opened 7 years ago

Closed 7 years ago

#15661 closed defect (fixed)

Integers have no order

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.1
Component: basic arithmetic Keywords:
Cc: Merged in:
Authors: Nathann Cohen Reviewers: Punarbasu Purkayastha
Report Upstream: N/A Work issues:
Branch: u/ncohen/15661 (Commits) Commit: 02493da12b082b381c8250a4efd73547fd351444
Dependencies: Stopgaps:

Description (last modified by mstreng)

Ahahahahaahah 

sage: 9.order()
+Infinity

This was deprecated by #5997. Let's remove it ;-)

Nathann

Change History (9)

comment:1 Changed 7 years ago by ncohen

  • Branch set to u/ncohen/15661
  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 7 years ago by git

  • Commit set to 02493da12b082b381c8250a4efd73547fd351444

Branch pushed to git repo; I updated commit sha1. New commits:

​02493datrac #15661: Integers have no order

comment:3 Changed 7 years ago by ncohen

  • Description modified (diff)

comment:4 Changed 7 years ago by ncohen

  • Description modified (diff)

comment:5 Changed 7 years ago by ppurka

  • Reviewers set to Punarbasu Purkayastha
  • Status changed from needs_review to positive_review 
----------------------------------------------------------------------
All tests passed!
----------------------------------------------------------------------
Total time for all tests: 506.8 seconds
    cpu time: 7079.0 seconds
    cumulative wall time: 7763.7 seconds
Version 0, edited 7 years ago by ppurka (next)

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

  • Description modified (diff)

I don't know why the deprecation warning mentions #5716. It was #5997. But that's still 5 years ago.

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

I don't know why the deprecation warning mentions #5716. It was #5997. But that's still 5 years ago.

Oh. Weird.

Well, either way I can only pray that there is nothing in the developper's manual about deprecating deprecation warnings when the ticket number is wrong :-p

Nathann

comment:8 Changed 7 years ago by kcrisman

This actually came up in the ask.sagemath questions about using lib gap recently, so good work.

comment:9 Changed 7 years ago by vbraun

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