-
Notifications
You must be signed in to change notification settings - Fork 714
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
Update the test requirements and dependencies, i.e. the MEM, WALLTIME, DEPENDENCY and DATA header items. #2300
Comments
I observed that they're still test scripts that do not specify any dependencies. The
|
For finding the dependencies of functions on other functions when refactoring the code, I have been using this in the past (which I dug up from my personal matlab folder on my laptop). It might also be somewhere in fieldtrip/private.
|
With f41a2f7 I have reduced the memory by 1gb or 2gb for many of the test scripts. I have kept the walltime mostly the same, except that I cleaned it up to have consistent "nice" numbers. There were also still a few scripts with errors in the formatting of the mem and wall time requirements. The dashboard script now includes a 2gb overhead for the memory, and a 30 minute overhead for the time. I believe that should allow all jobs to finish. It will probably need some tweaking later on. |
With #2310 some DATA dependencies were fixed and 3 new functions were added that will help to maintain the DEPENDENCY list up to date. |
Each test script should have these 4 header items. This has partially been done with #2278.
The text was updated successfully, but these errors were encountered: