Skip to content

Commit

Permalink
Declare astshim dependency in pybind11 code that uses it.
Browse files Browse the repository at this point in the history
Failing to do this could lead to it not being imported and some
wrappers not working.
  • Loading branch information
TallJimbo committed Mar 19, 2021
1 parent 2a5702d commit 1a5ff7a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions python/lsst/afw/geom/_skyWcs.cc
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,7 @@ void declareSkyWcs(lsst::utils::python::WrapperCollection &wrappers) {
void wrapSkyWcs(lsst::utils::python::WrapperCollection &wrappers) {
wrappers.addInheritanceDependency("lsst.afw.table.io");
wrappers.addInheritanceDependency("lsst.afw.typehandling");
wrappers.addSignatureDependency("astshim");
declareSkyWcs(wrappers);
}
} // namespace geom
Expand Down
1 change: 1 addition & 0 deletions python/lsst/afw/geom/_transform.cc
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ void declareTransform(lsst::utils::python::WrapperCollection &wrappers) {
} // namespace
void wrapTransform(lsst::utils::python::WrapperCollection &wrappers) {
wrappers.addSignatureDependency("lsst.afw.table.io");
wrappers.addSignatureDependency("astshim");
declareTransform<GenericEndpoint, GenericEndpoint>(wrappers);
declareTransform<GenericEndpoint, Point2Endpoint>(wrappers);
declareTransform<GenericEndpoint, SpherePointEndpoint>(wrappers);
Expand Down
1 change: 1 addition & 0 deletions python/lsst/afw/geom/_transformFactory.cc
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ void declareTransformFactory(lsst::utils::python::WrapperCollection &wrappers) {
} // namespace
void wrapTransformFactory(lsst::utils::python::WrapperCollection &wrappers) {
declareTransformFactory(wrappers);
wrappers.addSignatureDependency("astshim");
}
} // namespace geom
} // namespace afw
Expand Down

0 comments on commit 1a5ff7a

Please sign in to comment.