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
Tickets/dm 5503 #16
Tickets/dm 5503 #16
Conversation
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.
Good.
|
||
class MakeMegacamRawVisitInfo(MakeRawVisitInfo): | ||
"""Make a VisitInfo from the FITS header of a raw Megacam image | ||
""" |
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.
Supposed to have a blank line after class docstring.
@@ -124,8 +159,6 @@ def testPackageName(self): | |||
name = dafPersist.Butler.getMapperClass(root=self.repoPath).packageName | |||
self.assertEqual(name, "obs_cfht") | |||
|
|||
#-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- | |||
|
|||
|
|||
def suite(): |
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.
Would you mind upgrading this to support pytest while you're here?
and update testButler to make sure it works
and manually fix a few lines that needed more
Add `from __future__ import absolute_import, division, print_function` to every python module, or in one case move it to standard location
8372b0e
to
8e42247
Compare
I implemented your suggestions, ran autopep8 on all the python code and added the standard |
No description provided.