-
Notifications
You must be signed in to change notification settings - Fork 20
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
Creating a custom PCA background for Athena #31
Comments
You need to make sure that TELESCOP and INSTRUME header is consistent across files. Also, counts should be in integers. However, I don't know if really something bad happens if it isn't. |
Thanks for getting back to me so quickly! I tried adding the Also, after experimenting a bit the |
Maybe replace
with
|
and have a look what datasets contains |
Thanks! I replaced the Now I get a new error due to
From what I can work out, the condition |
do all your spectra have fewer counts than |
I am running BXA v2.10 on macOS Mojave v10.14.6, and am trying to create a new PCA model for the Athena/X-IFU from the publicly available background file (available from http://x-ifu.irap.omp.eu/resources/for-the-community) with the command:
python autobackgroundmodel/__init__.py bkg1.pha bkg2.pha bkg3.pha
First, the spectrum did not have a
COUNTS
column, so I created one with astropy based on the pre-existingRATE
column andEXPOSURE
header keywords. Since the spectrum appears to contain many counts, I wondered if it would be sufficient by itself to create a PCA model, instead of requiring multiple individual observations stacked together.I then ran the following command:
python3 /astrosoftware/BXA/autobackgroundmodel/__init__.py Total_pointsources_XIFU_CC_BASELINECONF_2018_10_10_COUNTS.pha
But I get the error message:
Is it possible this error is arising because I am using a much older version of BXA than the current one? If not, is there a way I can generate the PCA background for the X-IFU?
The text was updated successfully, but these errors were encountered: