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

Performance regression in unit test suite #131

Closed
al-maisan opened this issue Apr 9, 2011 · 5 comments
Closed

Performance regression in unit test suite #131

al-maisan opened this issue Apr 9, 2011 · 5 comments
Labels

Comments

@al-maisan
Copy link

The run time of the unit test suite quadrupled.

The following bash script was run last night on gemsun02 in order to exercise the test suite: http://paste.ubuntu.com/591606/.

According to the results (http://paste.ubuntu.com/591605/) the following change set caused the runtime of the unit test suite to quadruple:

commit bf18813219abd2ccae5254e9e92d9e6aa97982e8
Author: root <root@roundabout.(none)>
Date:   Fri Apr 8 08:33:20 2011 +0000

    Squashed commit of the following:

    commit b2bb2994e353c9f7e4b4d4ab55326bfed992a09d
    Author: Damiano Monelli <monelli.damiano@gmail.com>
    Date:   Mon Mar 21 16:44:04 2011 +0100

        added opensha-lite corrected for area source bug
@al-maisan
Copy link
Author

@ChristopherMacGown
Copy link
Contributor

git already has functionality to determine when a regression was introduced, please see the manpage for git bisect...

@al-maisan
Copy link
Author

Hello Chris, thanks for the hint -- will take a look.

@al-maisan
Copy link
Author

OK, had a look at git bisect. I don't think it's the right tool for this particular job since it is interactive in nature and automating it would probably take the same amount of time as writing the bash script I used in this instance.

Just my $ 0.02

@larsbutler
Copy link
Contributor

This is no longer an issue.

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

No branches or pull requests

3 participants