Skip to content

Adds calibration files - #24

Open
jajmitchell wants to merge 5 commits into
i4Ds:mainfrom
jajmitchell:adds_calibration_files
Open

Adds calibration files#24
jajmitchell wants to merge 5 commits into
i4Ds:mainfrom
jajmitchell:adds_calibration_files

Conversation

@jajmitchell

Copy link
Copy Markdown

This PR adds the necessary calibration files and STIX DRM file for the calibration of STIX data and the creation of an event specific SRM.

  • Reduce the size of the stix_drm_energy.npz file

Comment thread grid/grid_param_front.txt
29 0.077942 50.202 0.0629 0.0460 0.0011 0.0029 0.3597 0.0446 1.180051
30 0.160430 10.117 0.0747 0.0772 0.0007 0.0014 0.3903 0.0590 1.810433
31 0.228535 109.322 0.0951 0.1190 0.0006 0.0014 0.3838 0.0595 2.070182
32 0.682316 51.715 0.4884 0.3384 0.0018 0.0026 0.3980 0.0000 0.000000 No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Am blind or are these no identical ( I guess could be whitespace?)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unsure what you mean?

@samaloney samaloney Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was just looking a this file the matrix is 3230x3230 but the output count axis is only 31? Also I thought we now go above 150keV

hdul[3].data
Out[119]:
FITS_rec([(  4.,), (  5.,), (  6.,), (  7.,), (  8.,), (  9.,), ( 10.,),
          ( 11.,), ( 12.,), ( 13.,), ( 14.,), ( 15.,), ( 16.,), ( 18.,),
          ( 20.,), ( 22.,), ( 25.,), ( 28.,), ( 32.,), ( 36.,), ( 40.,),
          ( 45.,), ( 50.,), ( 56.,), ( 63.,), ( 70.,), ( 76.,), ( 84.,),
          (100.,), (120.,), (150.,)],
         dtype=(numpy.record, [('DRM', '>f4')]))

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I need to change the titles, this is the full output count axis, its used in the code when generating the correct SRM

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