This is a functional change in a pep8 PR :thumbsdown:

This might be ok, it might not be. I'd need to look at the code in more depth. For example masked arrays do interesting things with bools:
```
>>> import numpy.ma as ma
>>> a = ma.masked_array(23)
>>> a.mask
False
>>> a.mask == False
True
>>> a.mask is False
False
>>> type(a.mask)
<type 'numpy.bool_'>
```
To quote pep8
```
Don't compare boolean values to True or False using ==.

Yes:   if greeting:
No:    if greeting == True:
Worse: if greeting is True:
```