-
-
Notifications
You must be signed in to change notification settings - Fork 101
Improve CUTEst benchmarks #1323
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
Conversation
…ks are run" This reverts commit 5229720. Since the sandbox is run as a user and not root, packages cannot be installed due to insufficient permissions.
…ndling - Add chunked processing (50 problems per chunk) to manage memory usage - Implement comprehensive error handling with try/catch blocks - Add time limits (300s per problem) to prevent hanging - Force garbage collection between chunks to reduce memory pressure - Add detailed progress logging with chunk and problem tracking - Handle both problem loading and solving failures gracefully - Apply improvements to all CUTEst benchmark files: * CUTEst_bounded.jmd (666 + 244 problems) * CUTEst_unbounded.jmd (285 + 114 problems) * CUTEst_quadratic.jmd (252 problems) * CUTEst_unconstrained.jmd (286 problems) This resolves CI memory issues (ProcessSignaled(9)) while maintaining comprehensive testing of all CUTEst problem sets.
- Reduce chunk size from 5 to 3 problems per chunk - Lower variable limit from 100 to 50 variables per problem - Reduce maxiters from 1e6 to 1000 iterations - Keep maxtime at 60 seconds per problem - Add aggressive problem size filtering These changes should prevent ProcessSignaled(9) OOM errors in CI while still testing a substantial number of CUTEst problems.
|
Yes. I've tried updating the rootfs image and it seems to no longer complain about gfortran being missing.1 Footnotes |
c808547 to
ead6b8a
Compare
ead6b8a to
4b2d4d9
Compare
|
@arnavk23 it looks like it gets stuck. How long did that test set take locally? 2 days? |
|
No, it was done in around 3 hours. |
|
Did you run this same problem? All 6 succeeded? I don't see how because it had forward diff before and that is guaranteed to fail over the binaries |
|
|
|
Okay it has completely stalled with and now it's 3 days at that same spot. I think it's safe to say it's not printing anything else. Here's what we can do. We can merge this as it's now at least a major step forward in that it is actually able to run again, but it's still the same problem that it originally had so it's not complete. At least this makes it easier to work on though. |
|
@arnavk23 and you get 6 PDFs from that? |
|
@ChrisRackauckas I was able to get with the original AutoForwardDiff code only. |
|
@ChrisRackauckas I am still trying to see why this is stalling. |
I don't see how. It's not possible for forwarddiff the Fortran binaries, so you couldn't've been differentiating these examples. |
|
@ChrisRackauckas I saw you are doing this on Julia 10.10, try it instead on Julia 10.9 |
|
SciML benchmarks only runs on LTS forward. Current LTS is v10.10. But that also won't change this, fundamentally it's impossible for ForwardDiff to differentiate a fortran binary, it's not even possible in theory. |
|
@ChrisRackauckas , I meant the current one with FiniteDiff. Was looking through the code. |
|
Can you show the PDFs built with it? |
|
@ChrisRackauckas It is working, taking some time. Got unconstrained. Further it is your decision as to if I continue on this or not. |
|
That one ran before, but the issue there is most of them give failures. |
This is @arnavk23's commits rebased to latest master and setup as a PR to master.