Correction in list comprehension to allow the second term to be consider

[ANaaim]

No description provided.

[ANaaim]

Hi,

I think i found a small error in a list comprehension in your code :

    calib_dir = [os.path.join(project_dir, c) for c in os.listdir(project_dir) if ('Calib' or 'calib') in c][0]

Should be

    calib_dir = [os.path.join(project_dir, c) for c in os.listdir(project_dir) if ('Calib' in c or 'calib' in c)][0]

Elsewhere the second term is not taken into account :

I tested with the following code to be sure :

alpha = ['toto','tata','titi','tutu']
[c for c in alpha if ('tyty' or 'titi') in c]
Out[8]: []
[c for c in alpha if ('titi' or 'tyty') in c]
Out[9]: ['titi']
[c for c in alpha if ('titi' in c or 'tyty' in c)]
Out[10]: ['titi']
[c for c in alpha if ('tyty' in c or 'titi' in c)]
Out[11]: ['titi']

I hope it is correct.

Best regards,

[davidpagnon]

We will choose the second of your options, thanks!

[davidpagnon]

Oops wrong pull request.
I'll take care of this one as soon as the other one is done :)

[davidpagnon]

Interesting, thanks!