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-34863: Update tests and remove gen2 code from obs_decam. #227
Conversation
|
||
def getCrosstalkDir(): | ||
"""Get crosstalk directory in obs package for outputting data.""" | ||
return os.path.join(getPackageDir('obs_decam'), 'decam', 'crosstalk') |
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 think it's highly likely that you are going to want people to specify an input directory or a glob expanding to all the files rather than expecting the crosstalk files to always be here.
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.
This is the destination, it just puts them in decam/crosstalk
as before.
@@ -1,6 +1,5 @@ | |||
from .version import * | |||
from .decamMapper import * | |||
|
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.
Does this blank line need to be here?
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.
Maybe not, but I don't want to mess with the working ticket...
No description provided.