New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DM-15442: Add missing pyList=[] to test/SConscript files #60
Conversation
in scons and github Travis
d285e1d
to
4cd9210
Compare
I've turned on Travis. |
@@ -69,15 +65,15 @@ def run(visit, rerun, config): | |||
psf = butler.get('psf', dataId) | |||
print('PSF:', psf) | |||
|
|||
testDeblend(foots, pks, mi, psf) | |||
# testDeblend(foots, pks, mi, psf) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the code to test the deblender commented out?
examples/utils.py
Outdated
|
||
class _mockSource(object): | ||
class _MockSource(object): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could in theory remove (object)
here and elsewhere.
from builtins import zip | ||
from builtins import str | ||
from builtins import range | ||
from builtins import object | ||
#!/usr/bin/env python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't really need this shebang.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I also found and removed it in some other library and test files.
@@ -276,7 +279,7 @@ def deblend(self, exposure, srcs, psf): | |||
n0 = len(srcs) | |||
nparents = 0 | |||
for i, src in enumerate(srcs): | |||
#t0 = time.clock() | |||
# t0 = time.clock() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
time.clock()
is deprecated so it's debatable whether this code should be here at all. See DM-15450
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for catching this. I suggest we fix it on DM-15450
and stop using python 2 and enable automatic flake8 checking