Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ap_association, cat, ctrl_execute, ctrl_orca, ctrl_pool, daf_persistence, db, display_firefly, display_matplotlib, meas_modelfit, obs_base, obs_cfht, Qserv, sconsUtils, shapelet
-
Labels:None
-
Story Points:1.5
-
Sprint:Arch 2019-11-11, Arch 2019-11-18
-
Team:Architecture
Description
There are some packages in lsst_distrib that still use python future to support python 2.7. In some cases these are residual table file mentions or a handful of imports of past.builtins. Clean these up.
Attachments
Issue Links
- relates to
-
- Done
Activity
Kian-Tat Lim I've gone through all the lsst_distrib packages plus db, cat, and qserv, and removed all the python future usage (which includes importing things from builtins for no reason). I've also removed inheriting of object.
Some of these have been approved and merged by obvious candidates but a few still remain. Jenkins passed for lsst_distrib and Travis passes for qserv (I've asked Andy Salnikov to look at that one).
After this lsst_distrib only uses Python future for galsim and coord third party packages.
Chris Morrison [X] would you mind looking at the two line change to ap_association?
Steve Pietrowicz would you mind looking at ctrl_orca?
Tim Jenness I reviewed it, and the changes to ctrl_orca look fine.
Good riddance 
All PRs merged except for the firefly_client one. I'm leaving that for David Shupe to deal with since it's a fork of the upstream IPAC version.
Looking for:
from past.builtins import...from future.utils import...from future import...from __future__ import...also from builtins import is probably a porting crutch and we do not want to inherit from object.