Skip to content

DM-8449 resolve LSST catalog search panel running bugs #246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Dec 1, 2016

Conversation

cwang2016
Copy link
Contributor

@cwang2016 cwang2016 commented Nov 30, 2016

The development fixed the following issues:
1.Catalog polygon search doesn't work (wrong parameters passed)
2.Can not switch to catalog search after image search fails on creating the plot.
3.update the title of searched image table

@cwang2016
Copy link
Contributor Author

Test:
localhost:8080/firefly/lsst-pdac-triview.html

  1. select catalogs->Deep Source->target: 9.469, -1.152, method search: box, radius: 0.01 deg->search, then change method search to: polygon-> search.
  2. select image->Science CCD Exposure, target: 0, -1, size: 1 deg -> no image plot, then switch selection to catalogs and deep coadd, check spatial search panel for catalog
  3. select image -> Deep source -> target (0, -1) , size 1 deg, search,
    then target (1, -1), size 1 deg, search, check the title of searched image table in triview.

@ymeiymei
Copy link
Contributor

1 and 3 passed!

For Case 2. The image title for the search type "image covers entire search region" only shows the box size, no info about the RA and Dec. Need some fix?

@cwang2016
Copy link
Contributor Author

ra, dec numbers are added into the title for all search type of image table. (center, covers, and enclosed).

@ymeiymei
Copy link
Contributor

ymeiymei commented Dec 1, 2016

Test is passed. The RA and Dec information is added to the image title for all the search types.

Good job!

@cwang2016 cwang2016 merged commit de2e663 into dev Dec 1, 2016
@cwang2016 cwang2016 deleted the DM-8449-LSSTCatalogUI branch December 1, 2016 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants