smartdispatch icon indicating copy to clipboard operation
smartdispatch copied to clipboard

Published 20 hours ago • verified publisher icon SMART-Lab

Add verbosity to smart-dispatch

Open bouthilx opened this issue 7 years ago • 6 comments

It is difficult to debug resuming while important process are taking place in the pbs script automatically built by SmartDispatch.

Adding:

  • verbose to smart-dispatch/sd-launch-pbs command-line options
  • debugging prints in epilog (based on verbosity option)

bouthilx avatar Oct 18 '17 00:10 bouthilx

@mgermain Small PR I took out of Slurm Backend PR

bouthilx avatar Oct 18 '17 00:10 bouthilx

Coverage Status

Coverage remained the same at 94.532% when pulling 329efce4f71f799f9fdb4b3864ef53af65e94164 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

coveralls avatar Oct 18 '17 00:10 coveralls

Coverage Status

Coverage remained the same at 94.532% when pulling 502181da6d576160bcae129c04959f9d4860c532 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

coveralls avatar Oct 20 '17 19:10 coveralls

After the tiny modifs mentioned this is ready to be merged.

mgermain avatar Oct 20 '17 19:10 mgermain

Coverage Status

Coverage remained the same at 94.532% when pulling 65239f5700abcdeece5f76a15ff877ad211a2f72 on bouthilx:verbose into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

coveralls avatar Nov 03 '17 03:11 coveralls

@mgermain I bit the bullet and reverted the E501 fixes. :stuck_out_tongue_winking_eye: I also changed the debugging logic in the epilog to support passing WARNING/INFO/DEBUG levels to sd-launch-pbs. Previously, it could only pass WARNING (no -v) or DEBUG (-vv).

bouthilx avatar Nov 03 '17 03:11 bouthilx