Opened 10 years ago

Closed 10 years ago

#10969 closed enhancement (fixed)

Mark more doctests # long time

Reported by: jdemeyer Owned by: mvngu
Priority: major Milestone: sage-4.7
Component: doctest coverage Keywords: long time
Cc: Merged in: sage-4.7.alpha4
Authors: Jeroen Demeyer Reviewers: Florent Hivert
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

Some doctests take a long time, but are not marked # long time.

Sometimes, we can also restructure the doctest to make it take less time without essentially changing the test.

See also #10712.

Apply 10969_long_time.patch

Attachments (2)

10969_long_time.2.patch (15.9 KB) - added by jdemeyer 10 years ago.
10969_long_time.patch (20.7 KB) - added by jdemeyer 10 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 10 years ago by jdemeyer

  • Description modified (diff)

comment:2 follow-up: Changed 10 years ago by gagansekhon

Is this patch ready for review or are you still working on it?

comment:3 in reply to: ↑ 2 Changed 10 years ago by jdemeyer

  • Authors set to Jeroen Demeyer

Replying to gagansekhon:

Is this patch ready for review or are you still working on it?

I guess I might continue working on this.

Changed 10 years ago by jdemeyer

Changed 10 years ago by jdemeyer

comment:4 Changed 10 years ago by jdemeyer

  • Description modified (diff)
  • Status changed from new to needs_review

comment:5 Changed 10 years ago by hivert

  • Reviewers set to Florent Hivert
  • Status changed from needs_review to positive_review

Patch looks good ! Ready to go !

comment:6 Changed 10 years ago by jdemeyer

  • Merged in set to sage-4.7.alpha4
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.