# Remove CatalogStarSelector

XMLWordPrintable

## Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
2
• Team:

## Activity

Hide
Chris Morrison added a comment -
Show
Chris Morrison added a comment - Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29260/pipeline
Hide
John Parejko added a comment -

A bulk search for catalogStar turns up a mention in a docstring for pipe_tasks/processCcd.py and an if statement in ip_diffim/diaCatalogSourceSelector.py that should both be cleaned up. I think that's it?

Show
John Parejko added a comment - A bulk search for catalogStar turns up a mention in a docstring for pipe_tasks/processCcd.py and an if statement in ip_diffim/diaCatalogSourceSelector.py that should both be cleaned up. I think that's it?
Hide
John Parejko added a comment -

Thanks for the removals. Please file that RFC to cleanup this interface to processCcd.

Show
John Parejko added a comment - Thanks for the removals. Please file that RFC to cleanup this interface to processCcd.
Show
Chris Morrison added a comment - Final Jenkins just in case: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/29272/pipeline/46/
Hide
Chris Morrison added a comment -

Considering my lack of familiarity with that particular code path or even why it was there in the first place, I will let someone else file the RFC Jim Bosch

Show
Chris Morrison added a comment - Considering my lack of familiarity with that particular code path or even why it was there in the first place, I will let someone else file the RFC Jim Bosch
Hide
Jim Bosch added a comment -

I may not do it immediately, but I will be looking at that code next week anyway and that should give me an opportunity to say more precisely what we'd want to do.

Show
Jim Bosch added a comment - I may not do it immediately, but I will be looking at that code next week anyway and that should give me an opportunity to say more precisely what we'd want to do.

## People

• Assignee:
Chris Morrison
Reporter:
Chris Morrison
Reviewers:
John Parejko
Watchers:
Chris Morrison, Jim Bosch, John Parejko, John Swinbank