-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
2x regression in testing time of small package #53511
Comments
Adding a |
I think a big reason for the slowdown in the end to end testing is that it seems that precompiled code is turned off for stdlibs when running with code coverage now?
|
Yeah. But even with the path approach, as soon as a package is loaded within that path, any further loads don't use pkgimage native code either. The fix is to load your test package last in the test suite. |
Perhaps Aqua should check that the test package is loaded last |
I added some data in KristofferC/LazilyInitializedFields.jl#38. The last commit (loads the testing package last but it doesn't seem to have much effect. |
I'm going to fold this one into #53570 |
Looking at the time to take to test a simple package (https://github.com/KristofferC/LazilyInitializedFields.jl/actions/runs/8072420307/job/22079073318?pr=30) I see an over 2x regression end to end.
Some numbers:
It is possible this is mostly the same as #52592 which was closed, first without a comment and then with
but a 2x regression end to end for a very standard simple package seems like something that should be solved
The text was updated successfully, but these errors were encountered: