Closed calvinp0 closed 1 year ago
Merging #640 (acaadf7) into main (8b1a8ee) will decrease coverage by
0.01%
. The diff coverage is40.00%
.:exclamation: Current head acaadf7 differs from pull request most recent head 010848e. Consider uploading reports for the commit 010848e to get more accurate results
@@ Coverage Diff @@
## main #640 +/- ##
==========================================
- Coverage 73.14% 73.14% -0.01%
==========================================
Files 99 99
Lines 26277 26276 -1
Branches 5495 5495
==========================================
- Hits 19220 19219 -1
- Misses 5694 5695 +1
+ Partials 1363 1362 -1
Impacted Files | Coverage Δ | |
---|---|---|
arc/job/local.py | 40.54% <0.00%> (-0.67%) |
:arrow_down: |
arc/job/adapter.py | 40.50% <100.00%> (ø) |
|
arc/job/adapter_test.py | 98.11% <100.00%> (ø) |
... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR ensures that if Max Server requests are reached, then the software will sleep for 5 minutes and then attempt to resubmit again It also fixes the issue of PBS memory setting furthermore, if the user provides memory in as a string, such as '34G' then the software will allow it