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-14842: Fix deprecation warnings from PropertyList/Set.get #10
Conversation
Use `getScalar` or `getArray` instead of deprecated `get` on `PropertySet` and `PropertyList` instances.
python/lsst/synpipe/compareModel.py
Outdated
@@ -103,8 +100,7 @@ def main(root, visit, ccds, galType='sersic', output='outputs/'): | |||
|
|||
npTable = writeNumpyTable(fakeTable) | |||
rerunName = root.split('/')[-2] | |||
fitsTable = astropy.table.Table(npTable).write(output+'/'+rerunName + | |||
'_galMags.txt', format='ascii') | |||
astropy.table.Table(npTable).write(output+'/'+rerunName + '_galMags.txt', format='ascii') |
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.
Add some extra spaces around the + while you're already editing this line. (Looks like it's not approaching 110)
@@ -1,10 +1,5 @@ | |||
#!/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.
Why does this go away and not in python/lsst/synpipe/makeFakeGalaxy.py
too?
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.
I must have missed that one. I'll fix it and a bunch of others.
Note that many of these same modules have if __name__ == "__main__":
sections, which don't belong in library code. But I'm leaving them for now.
And make one line easier to read by breaking it into two lines
Add a minimal unit test that imports the library and checks for the presence of a symbol. A unit test is necessary in order to have scons run flake8 reliably and an import test is better than nothing.
I fixed the two issues you raised |
both by scons and by Travis on github
No description provided.