Skip to content

Commit

Permalink
Add a note about a strange py wrapper import
Browse files Browse the repository at this point in the history
The simple.cc pybind11 wrapper imports lsst.afw.coord.
It should not be necessary, but removing it causes problems
so I made a note with a reference to a possibly relevant RFC.
  • Loading branch information
r-owen committed Mar 22, 2018
1 parent 780a65c commit 7de6782
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion python/lsst/afw/table/simple/simple.cc
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ PySimpleTable declareSimpleTable(py::module &mod) {
} // namespace lsst::afw::table::<anonymous>

PYBIND11_PLUGIN(simple) {
py::module::import("lsst.afw.coord");
py::module::import("lsst.afw.coord"); // needed, but why? Perhaps can be removed after RFC-460
py::module::import("lsst.afw.table.base");
py::module::import("lsst.afw.table.idFactory");

Expand Down

0 comments on commit 7de6782

Please sign in to comment.