AgnostiqHQ / covalent-slurm-plugin

Executor plugin interfacing Covalent with Slurm
https://covalent.xyz
Apache License 2.0
27 stars 6 forks source link

Clean up and improve error handling #86

Closed araghukas closed 8 months ago

araghukas commented 9 months ago

Updated the plugin with the following changes:

Notes


codecov[bot] commented 8 months ago

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (f5085fb) 93.16% compared to head (429f0c6) 92.00%.

Files Patch % Lines
covalent_slurm_plugin/exec.py 91.48% 4 Missing :warning:
covalent_slurm_plugin/job_script.py 96.00% 3 Missing :warning:
covalent_slurm_plugin/slurm.py 97.50% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #86 +/- ## =========================================== - Coverage 93.16% 92.00% -1.17% =========================================== Files 2 4 +2 Lines 278 375 +97 =========================================== + Hits 259 345 +86 - Misses 19 30 +11 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.