122 sbatch

[abhidg]

• ci: update pre-commit hooks
• feat: add sbatch scheduler support, fixes scheduler: add sbatch #122

[codecov]

Codecov Report

Attention: Patch coverage is 81.08108% with 7 lines in your changes missing coverage. Please review.

Project coverage is 87.25%. Comparing base (81d44d0) to head (5f2831a).

Files with missing lines	Patch %	Lines
cats/__init__.py	56.25%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   87.20%   87.25%   +0.05%     
==========================================
  Files          14       14              
  Lines         586      612      +26     
==========================================
+ Hits          511      534      +23     
- Misses         75       78       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[andreww]

This all looks okay to me. Has it been tested in reality on a system running slurm?

[sadielbartholomew]

@abhidg do you need a second review or is one sufficient?

[abhidg]

@sadielbartholomew could you check if this PR works in a cluster? I don't have an account on our supercomputer ATM.

[sadielbartholomew]

could you check if this PR works in a cluster? I don't have an account on our supercomputer ATM.

Sure, as long as you don't mind waiting a couple of days - I have some urgent work to sort for tomorrow and it will take a bit more time to review on JASMIN or ARCHER2. Will aim to submit a cluster-based review by the end of the week.