Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: obs_sdss
-
Labels:None
-
Team:Data Release Production
Description
Installing lsst_apps -t w_2015_45 fails with
ERROR: ImportError: cannot import name six [unknown]
|
Traceback (most recent call last):
|
File "tests/getId.py", line 29, in <module>
|
from lsst.obs.sdss import SdssMapper
|
File "/Users/lsst/products/EupsBuildDir/DarwinX86/obs_sdss-11.0-3-g1664b15+1/obs_sdss-11.0-3-g1664b15+1/python/lsst/obs/sdss/__init__.py", line 24, in <module>
|
import makeCamera
|
File "/Users/lsst/products/EupsBuildDir/DarwinX86/obs_sdss-11.0-3-g1664b15+1/obs_sdss-11.0-3-g1664b15+1/python/lsst/obs/sdss/makeCamera.py", line 30, in <module>
|
from lsst.obs.sdss.convertOpECalib import SdssCameraState
|
File "/Users/lsst/products/EupsBuildDir/DarwinX86/obs_sdss-11.0-3-g1664b15+1/obs_sdss-11.0-3-g1664b15+1/python/lsst/obs/sdss/convertOpECalib.py", line 4, in <module>
|
from lsst.obs.sdss.yanny import yanny as Yanny
|
File "/Users/lsst/products/EupsBuildDir/DarwinX86/obs_sdss-11.0-3-g1664b15+1/obs_sdss-11.0-3-g1664b15+1/python/lsst/obs/sdss/yanny.py", line 35, in <module>
|
from astropy.extern import six
|
ImportError: cannot import name six
|
Attachments
Issue Links
- relates to
-
- Done
Activity
Usage of astropy has snuck in via an update to Yanny (DM-4032); it should be easy to remove as it's not used much.
John Swinbank points out that astropy is also used in daf_butlerUtils (DM-4170).
These uses of astropy either need to be approved via the RFC process, or removed.
Paul Price
added a comment - Usage of astropy has snuck in via an update to Yanny ( DM-4032 ); it should be easy to remove as it's not used much.
John Swinbank points out that astropy is also used in daf_butlerUtils ( DM-4170 ).
These uses of astropy either need to be approved via the RFC process, or removed. | Field | Original Value | New Value |
|---|---|---|
| Team | Data Release Production [ 10301 ] |
We don't notice these things because astropy was added to the Jenkins dependency tree for Sims and newinstall.sh uses anaconda (which comes with astropy). (Joshua Hoblitt will be mumbling "I told you so" at this point).
Oog. Sorry about that! The intention was not to bring in astropy in this case. I forgot that six is an astropy.extern package. I do wonder why it didn't fail in my original jenkins build, which I thought was before astropy was added to Jenkins.
I can try to clean out the six references: there's only a couple.
I'll sort this out, since I made the mess.
| Assignee | John Parejko [ parejkoj ] |
Should be fixed in tickets/DM-4361 branch. Can someone else please check to be sure?
Since you discovered it, can you please test it?
| Reviewers | Robert Lupton [ rhl ] | |
| Status | To Do [ 10001 ] | In Review [ 10004 ] |
Anyone object to the merge request?
Fine with me.
Changes merged, builds should not have trouble with missing astropy now.
| Resolution | Done [ 10000 ] | |
| Status | In Review [ 10004 ] | Done [ 10002 ] |
Is this breakage from
DM-4032? Looks like the new yanny reader brought in an undisclosed astropy dependency. Not good.