Fix Version/s: None
Component/s: obs_subaru, Validation
When attempting to reprocess some hsc data, I ran into a number of problems (one fixed in
DM-8822). In this case, some of the python type definition in the obs_subaru Mapper.paf files are not valid: __builtin__.str doesn't exist on python3. Fortunately because of our use of futurize, we have builtins.str.
Although I don't plan to write it as part of this ticket, having a python2+3 validation system for the policy files would be good.
This looks fine, but please ensure it passes ci_hsc (it's now fixed) before merging.
Thanks for the ci_hsc fix. Butler run succeeded after a rebase:
I still want to hear from Nate Pease [X] about whether and how they dealt with this in other products.
I haven't dealt with _builtin_.str in policy files before. This solution seems fine to me, if it's passing tests on Jenkins in py2 and py3.
Thanks for the comments.
As I noted above, ci_hsc passed Jenkins on py2, but ci_hsc doesn't run on py3 yet, so the only "proof" that this worked is that I was able to reprocess hsc data with a py3 stack. We may have to revisit this issue during the py3 port of ci_hsc.
Merged and done.
I don't know, but if you put "ci_hsc" in jenkins, it fails in a couple of minutes. I haven't tried running it locally.