-
Notifications
You must be signed in to change notification settings - Fork 7
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-45325: Move CTI simulation classes to lsst.ip.isr.deferredCharge #261
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.
This is a little bit tricky, but we need to deprecate the old imports in case somebody has been lifting OverscanModel
from here. See https://developer.lsst.io/stack/deprecating-interfaces.html in general, and see https://github.com/lsst/pipe_tasks/blob/c100eb463524ac2dc860a101d48d552f65cd4135/python/lsst/pipe/tasks/assembleCoadd.py#L32 for a possible pattern to lift the imports and warn if somebody imports from the old location. You also need to file a ticket to remove the deprecated usage and add that as a TODO.
f0e16d3
to
88b57a4
Compare
88b57a4
to
a2a542a
Compare
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.
The __init__.py
needs a final newline. This isn't triggering the linter because we generally skip __init__.py
(because they have unused imports).
a2a542a
to
1863454
Compare
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.
Looks good pending the formatting problem on the __init__
1863454
to
782302c
Compare
782302c
to
51a0e00
Compare
No description provided.