Opened 11 years ago

Last modified 6 years ago

#12722 new enhancement

Add the option to doctest a file in parallel

Reported by: David Roe Owned by: David Roe
Priority: major Milestone: sage-wishlist
Component: doctest framework Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #9461 Stopgaps:

Status badges

Description

There are some files in Sage where doctesting that file can take more than a minute. It would be nice to be able to take advantage of multiple cores when making changes to such a file.

In order to allow parallel doctesting to succeed, #12721 is needed. But adding the option to the doctest module doesn't depend on #12721 being resolved.

Change History (7)

comment:1 Changed 10 years ago by Jeroen Demeyer

Dependencies: #12415#12415, #12721

comment:2 Changed 10 years ago by Jeroen Demeyer

I don't really see the advantage of being able to do this. This will just add another layer of complexity to the doctesting framework for a small advantage. And I think even #12721 will already be difficult to fix.

comment:3 in reply to:  2 Changed 10 years ago by David Roe

Replying to jdemeyer:

I don't really see the advantage of being able to do this. This will just add another layer of complexity to the doctesting framework for a small advantage. And I think even #12721 will already be difficult to fix.

The advantage is if you're making changes to one file that takes 60 seconds to doctest, it's nice to have the turnaround time faster.

I agree that it's not a big benefit, and I don't really mind closing it as wontfix given the additional complexity in the doctesting framework. But there is a benefit for users (especially new ones) who are just editing one file.

comment:4 Changed 10 years ago by David Roe

Component: doctestdoctest framework
Owner: changed from Minh Van Nguyen to David Roe

comment:5 Changed 10 years ago by Jeroen Demeyer

Milestone: sage-5.9sage-wishlist

Moving to "wishlist" as I don't think it's realistic that it will be fixed.

comment:6 in reply to:  5 Changed 10 years ago by David Roe

Replying to jdemeyer:

Moving to "wishlist" as I don't think it's realistic that it will be fixed.

Sure. I think it's doable but I don't care enough to do it. :-)

comment:7 Changed 6 years ago by Jeroen Demeyer

Dependencies: #12415, #12721#9461
Note: See TracTickets for help on using tickets.