Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: butler, pipe_drivers
-
Labels:None
-
Story Points:2
-
Epic Link:
-
Team:Data Release Production
Description
See https://community.lsst.org/t/access-to-root-in-butler/991
When running constructBias.py in lsst_apps w_2016_34 I get the error:
AttributeError in map: 'Butler' object has no attribute 'repository'
The source of the error is probably
https://github.com/lsst/pipe_drivers/blob/master/python/lsst/pipe/drivers/constructCalibs.py#L585
All code that uses butler.repository should be fixed. If that's too much work possibly the ticket could get changed to fix constructCalibs only for now.
Attachments
Issue Links
- blocks
-
- Done
Activity
Removed the butler.repository line
Andreas Ritter
added a comment - Removed the butler.repository line The fix is fine.
Did we capture the desire to save this information? It isn't available from the butler for the reasons given, but I think that the script (constructCalibs.py) does know its root – is that right, Paul Price? If so, couldn't we get it from there?
Note that it's not the butler root directory you want, but the calibs root directory. I think that should be available from the argument parser, but you may have to apply the same defaults logic as in the butler.
Paul Price
added a comment - Note that it's not the butler root directory you want, but the calibs root directory. I think that should be available from the argument parser, but you may have to apply the same defaults logic as in the butler. This has been merged to master.
Paul Price
added a comment - This has been merged to master. 
Absolutely fine with me