Paul Price added a comment - 02/Sep/16 12:27 PM Lauren MacArthur , would you mind having a look at this? Bob Armstrong already reviewed it on the HSC side, but there were some conflicts in the cherry-pick. price@price-laptop:~/LSST/obs/subaru (tickets/DM-7510=) $ git sub commit ff0ccdaec844f1ddbaaf23a186cd94943e17f435 Author: Paul Price <price@astro.princeton.edu> Date: Wed Aug 31 14:33:50 2016 -0400   add support for HSC-R2 filter Color terms provided by Yasuda-san.   config/hsc/colorterms.py | 2 ++ config/hsc/filterMap.py | 2 +- config/hsc/mergeCoaddDetections.py | 5 ++--- config/hsc/mergeCoaddMeasurements.py | 5 ++--- python/lsst/obs/hsc/hscMapper.py | 2 ++ 5 files changed, 9 insertions(+), 7 deletions(-)

Lauren MacArthur added a comment - 02/Sep/16 12:34 PM Sure thing. Any idea why the branch doesn't show up in Jira?

Paul Price added a comment - 02/Sep/16 12:37 PM I made a PR, and it's showing up now.

Lauren MacArthur added a comment - 02/Sep/16 12:57 PM Is there any HSC-R2 data anywhere on tiger I could test this on? There's nothing in /tigress/HSC/HSC/registry.sqlite3 . Otherwise, it looks like you could add r2 to the list here: https://github.com/lsst/obs_subaru/blob/master/tests/testColorterms.py#L73

Paul Price added a comment - 02/Sep/16 1:39 PM I don't have any R2 data that I'm aware of yet. I added r2 to the test.

Paul Price added a comment - 02/Sep/16 1:39 PM Jenkins passed .

Lauren MacArthur added a comment - 02/Sep/16 1:45 PM Ok. I don't think it can matter in this case, but in general would it be a good idea to include ci_hsc in your Jenkins build when you've made edits in obs_subaru ? Good to go.

Paul Price added a comment - 07/Sep/16 7:51 AM Thanks Lauren. I think I had already tested through ci_hsc previously, so when I expanded the test I only went through obs_subaru. Merged to master.