Skip to content

Commit

Permalink
Change order of constructors for lsst.afw.image.Calib
Browse files Browse the repository at this point in the history
Put the PropertySet in front of list[Calib] since PropertySet
now has __len__ and __getitem__ methods and so pybind11 thinks
that it is probably a sequence.
  • Loading branch information
timj committed Sep 11, 2018
1 parent 375968b commit 0666c60
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion python/lsst/afw/image/calib.cc
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,8 @@ PYBIND11_MODULE(calib, mod) {
/* Constructors */
cls.def(py::init<>());
cls.def(py::init<double>(), "fluxMag0"_a);
cls.def(py::init<std::vector<std::shared_ptr<const Calib>> const &>(), "calibs"_a);
cls.def(py::init<std::shared_ptr<const daf::base::PropertySet>>(), "metadata"_a);
cls.def(py::init<std::vector<std::shared_ptr<const Calib>> const &>(), "calibs"_a);

table::io::python::addPersistableMethods<Calib>(cls);

Expand Down

0 comments on commit 0666c60

Please sign in to comment.