psathyrella / partis

B- and T-cell receptor sequence annotation, simulation, clonal family and germline inference, and affinity prediction
GNU General Public License v3.0
54 stars 34 forks source link

Confusing n procs status messages #272

Closed scharch closed 5 years ago

scharch commented 5 years ago

I consistently get messages like running 9 procs for 108999 seqs where the number is greater than I specified at the command line (in this case: --n-procs 6). I don't think this is a bug in resource requests, though, as when I look at System Monitor, I see 6 processes (1 python + 5 ig-sw), so I assume what this means is that I have enough sequences that partis wants to split them into 9 pieces, even though only five will be run at once. If I'm right, may I suggest clarifying the message?

psathyrella commented 5 years ago

The short answer is that in some cases I'm taking --n-procs more as a suggestion than an order ;-)

Just going by memory here (I'll look through and see if I can make it more clear what's happening), but I think there's two cases where this is going to happen:

psathyrella commented 5 years ago

I added a note about this to the help for --n-procs. If you see it hammer the machine with an absurd number of processes, that's a bug, but as long as it's like 9 vs 6, then that's expected behavior.