Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add speed regression database to doctesting #12720

Closed
roed314 opened this issue Mar 21, 2012 · 12 comments
Closed

Add speed regression database to doctesting #12720

roed314 opened this issue Mar 21, 2012 · 12 comments

Comments

@roed314
Copy link
Contributor

roed314 commented Mar 21, 2012

Currently there's no systematic way of detecting whether a change to the sage library drastically slows down some function (perhaps even the function that's being updated in a ticket).

This ticket aims to add features to the new doctesting code at #12415 that provide the option to run doctests in a mode where they add timing data to a database.

Apply attachment: 12720.patch

Apply attachment: 12720_script.patch to the script repo

CC: @videlec @koffie @embray @roed314

Component: doctest framework

Issue created by migration from https://trac.sagemath.org/ticket/12720

@jhpalmieri
Copy link
Member

comment:1

Could monitoring Sage's startup time be incorporated into this?

@roed314
Copy link
Contributor Author

roed314 commented Mar 15, 2013

Changed dependencies from #12415 to #13278

@roed314
Copy link
Contributor Author

roed314 commented Mar 15, 2013

Still very much in progress

@roed314
Copy link
Contributor Author

roed314 commented Mar 15, 2013

Attachment: 12720.patch.gz

@roed314
Copy link
Contributor Author

roed314 commented Mar 15, 2013

comment:3

Attachment: 12720_script.patch.gz

@roed314

This comment has been minimized.

@roed314
Copy link
Contributor Author

roed314 commented Mar 15, 2013

comment:4

Now that #12415 is in Sage, I'll be working on this some. It will probably be at least a few weeks though (I'm going to be working on p-adics and the git transition first).

In the mean time, I've updated the patches to apply against #12415 (actually, rebased against #13278). There's still a lot to be done to make them work well.

@roed314 roed314 assigned roed314 and unassigned sagetrac-mvngu Mar 28, 2013
@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@videlec
Copy link
Contributor

videlec commented Apr 13, 2015

comment:10

Hello,

just cc'ing me!

@jdemeyer
Copy link

Changed dependencies from #13278 to #24655

@jdemeyer jdemeyer modified the milestones: sage-6.4, sage-8.2 Apr 29, 2018
@saraedum
Copy link
Member

comment:13

roed, I believe that this should probably be closed. The approach of rolling our own system for doing something that asv does very well is probably not appropriate anymore. What do you think?

@jdemeyer
Copy link

comment:14

Duplicate of #25262

@jdemeyer
Copy link

Changed dependencies from #24655 to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants