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

specextract response position when background has multiple shapes #732

Open
kglotfelty opened this issue Jan 18, 2023 · 1 comment
Open

Comments

@kglotfelty
Copy link
Member

kglotfelty commented Jan 18, 2023

We are getting a Warning when the background contains multiple shapes when resp_pos=REGION, but we shouldn't. The response position should only be affected by the source region, not the background.

% download_chandra_obsid 4425 evt2,asol,bpix,msk
% cat src.reg
circle(4061.5548,4036.5351,5.1111071)
% cat bkg.reg
circle(4021,4023,23.465767)
circle(4091,4077,29.702094)
% punlearn specextract
% specextract "4425/primary/acisf04425N005_evt2.fits.gz[sky=region(src.reg)]" \
  bkgfile="4425/primary/acisf04425N005_evt2.fits.gz[sky=region(bkg.reg)]" \
  out=foo weight- bkgresp- mode=h clob+
Running specextract
Version: 18 November 2022

Warning: more than 1 shape found, using the coordinates of the first defined shape
Extracting Spectra:[############################################################] 2/2

Generating Aspect Histograms & Weights Maps:[############################################################] 2/2

Generating ARFs & RMFs:[############################################################] 4/4

Grouping Spectra:[############################################################] 2/2

Adding Header Keywords:[############################################################] 2/2

It looks to be benign, but just confusing. (Helpdesk ticket 24444)

@nplee
Copy link
Contributor

nplee commented Jan 18, 2023

fixed... or at least the message is quashed for the background/no background response case, since the function's still used to get the RA and Dec to estimated a hydrogen column density.

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

No branches or pull requests

2 participants