Update htcondor.py

[steog88]

I am proposing this change after some attempts working with the parallel queue in the Torino Cloud inftastructure. Nodes that require MPI and run on several virtual machines were not correctly treated by the previous code.

[dberzano]

Hello @steog88, thanks for your contribution. I kindly ask you to address a number of points before merging this PR.

• Please get rid of all the commented debug lines by completely deleting the lines.
• Use spaces as the rest of the code does: item, item, item += item, item = item for instance
• Why using an expensive regular expression to search a simple string. Do "requireswholemachine = true" in j.lower() which is much more pythonic
• You can force n_hosts to float only once in your code (instead of twice)
• Please get rid of the parentheses in the ifs (i.e. no if (a == b) but if a == b)
• Rename out to something more meaningful (as n_waiting)

I like the -constraint which would make my original code simpler 👍

From what I see this should work both for the "whole machine" case and the non-whole machine one. Did you test it to make sure it's the case?

(This morning I woke up in --pedantic mode, sorry.)

[steog88]

Hi, thanks for the suggestions.
I hope to have addressed all the requirements.

Yes, I tested it with and without the "WholeMachine" case and it works.