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-9190: Cleanup pybind11 remaining code #68
Conversation
510fdf6
to
0f733de
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.
The obs_subaru code could use a cleanup pass (e.g. rename Crosstalk.h
to subtractCrosstalk.h
to avoid collision with crosstalk.py
, fix pep8 violations) but that seems out of scope.
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 missed one missing import the first time.
python/lsst/obs/subaru/__init__.py
Outdated
from .subaruLib import * | ||
from __future__ import absolute_import, division, print_function | ||
|
||
import lsst.afw.geom |
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.
Consider moving this to hscDistortion.cc
python/lsst/obs/subaru/_crosstalk.cc
Outdated
@@ -31,6 +31,8 @@ namespace lsst { | |||
namespace obs { | |||
namespace subaru { | |||
|
|||
// Note that _crosstalk is not related to the crosstalk Python module; | |||
// unfortunately they do have the same name. | |||
PYBIND11_PLUGIN(_crosstalk) { | |||
py::module mod("_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.
Please import lsst.afw.image (here or in __init__.py
)
0f733de
to
6d1c961
Compare
6d1c961
to
d2c8f16
Compare
No description provided.