search

European-XFEL / DAMNIT

Data And Metadata iNspection Interactive Thing
https://damnit.rtfd.io
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

Lower the default slurm concurrency limit to 15 #339

Closed JamesWrigley closed 2 weeks ago

JamesWrigley commented 2 weeks ago

With 30 I still observe some SQLite locking errors, including from the GUI (which crashed it).

takluyver commented 2 weeks ago

If we're changing the default, let's change it for the CLI as well:

https://github.com/European-XFEL/DAMNIT/blob/2548d5ebe39ee9e89beb6dd5268e9d151162ea1f/damnit/cli.py#L112-L115

Other than that, LGTM.

JamesWrigley commented 2 weeks ago

Roger that, fixed in 5ebdc47.