Skip to content
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

Imviz: Clarify how subset is extracted when multiple data are loaded (was: explicit layer in subset plugin) #1629

Open
Jdaviz-Triage-Bot opened this issue Sep 8, 2022 · 6 comments
Labels

Comments

@Jdaviz-Triage-Bot
Copy link

Jdaviz-Triage-Bot commented Sep 8, 2022

Reporter: Camilla Pacifici

Update: See #1629 (comment)

Here is the workflow:

  • load two images with overlapping regions, but not on same field of view
    (mast:JWST/product/jw01345-o002_t022_nircam_clear-f200w_i2d.fits
    mast:JWST/product/jw01345-o003_t023_nircam_clear-f200w_i2d.fits)
  • select subset in first image in the overlapping region
  • check position of the same subset in pixels in the second image
    expected that it is not covering the same object
  • link by WCS
  • now subset covers the same ra-dec in the two images
  • look at image 2 and open subset plugin
    the pixels in the subset do not match what seeing in the image
    this is confusing
  • open aperture photometry plugin
    get results with subset on image 1
    results make sense
  • blink to image 2
  • use photometry plugin with same subset in image 2
    subset still looks somewhere else
    results do not make sense even if I see the subset on the right object

DISCLAIMER: This issue was autocreated by the Jdaviz Issue Creation Bot on behalf of the reporter. If any information is incorrect, please contact Duy Nguyen

@camipacifici
Copy link
Contributor

Might be fixed by #1524 ... testing

@camipacifici
Copy link
Contributor

Yes! #1524 fixes the photometry problem here.

The confusing output of the subset plugin is still valid. Maybe the subset plugin should also mention the layer where the subset has been created?

@camipacifici camipacifici changed the title Imviz: fix workflow for photometry on images with partial overlap Imviz: explicit layer in subset plugin Sep 8, 2022
@pllim pllim added the imviz label Sep 8, 2022
@pllim
Copy link
Contributor

pllim commented Sep 8, 2022

Maybe the subset plugin should also mention the layer where the subset has been created?

That might be a bit tricky but maybe @astrofrog can clarify. As far as I can understand, Subset extraction is based on what happens to be the reference data at the time of extraction, and reference data is tied to a particular viewer and can be changed in some circumstances.

@pllim
Copy link
Contributor

pllim commented Sep 8, 2022

p.s. I am going to clarify the Issue title.

@pllim pllim changed the title Imviz: explicit layer in subset plugin Imviz: Clarify how subset is extracted when multiple data are loaded (was: explicit layer in subset plugin) Sep 8, 2022
@kecnry
Copy link
Member

kecnry commented Sep 8, 2022

As far as I can understand, Subset extraction is based on what happens to be the reference data at the time of extraction, and reference data is tied to a particular viewer and can be changed in some circumstances.

So similar to how we display the top-layer being referenced in the compass plugin, couldn't we have an indicator showing the current reference layer?

@pllim
Copy link
Contributor

pllim commented Sep 8, 2022

couldn't we have an indicator showing the current reference layer?

We could, but I am not sure about live-updating the Subset plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants