Opened 8 years ago

Last modified 3 years ago

#12722 new enhancement

Add the option to doctest a file in parallel

Reported by: roed Owned by: roed
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:

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 7 years ago by jdemeyer

  • Dependencies changed from #12415 to #12415, #12721

comment:2 follow-up: Changed 7 years ago by 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.

comment:3 in reply to: ↑ 2 Changed 7 years ago by roed

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 7 years ago by roed

  • Component changed from doctest to doctest framework
  • Owner changed from mvngu to roed

comment:5 follow-up: Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.9 to sage-wishlist

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

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

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 3 years ago by jdemeyer

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