-
Notifications
You must be signed in to change notification settings - Fork 40
Corrections for Lo Maps #3370
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
base: dev
Are you sure you want to change the base?
Corrections for Lo Maps #3370
Changes from all commits
8f56b85
a7fe482
304a808
42a1b27
bda2ff5
958a8a2
87b95f9
1674dea
570004b
45d20bc
6e4a22f
b5346ab
6fefb19
e63f74b
7d0cd63
5e8b085
e5f1cc9
1d03362
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| pivot_angle,esa_step,intensity_threshold_fraction,angular_width_deg,outlier_percentile | ||
| 75,1,0.015,80.0,99.999 | ||
| 75,2,0.015,70.0,99.999 | ||
| 75,3,0.015,70.0,99.999 | ||
| 75,4,0.008,70.0,99.999 | ||
| 75,5,0.02,25.0,96.999 | ||
| 75,6,1.0,35.0,99.999 | ||
| 75,7,1.0,35.0,99.999 | ||
| 90,1,0.01,80.0,99.999 | ||
| 90,2,0.01,70.0,99.999 | ||
| 90,3,0.005,65.0,99.999 | ||
| 90,4,0.005,50.0,99.999 | ||
| 90,5,0.1,45.0,99.999 | ||
| 90,6,1.0,35.0,99.999 | ||
| 90,7,1.0,35.0,99.999 | ||
| 105,1,0.01,80.0,99.999 | ||
| 105,2,0.005,60.0,99.999 | ||
| 105,3,0.001,60.0,99.999 | ||
| 105,4,0.005,40.0,99.999 | ||
| 105,5,0.15,35.0,99.999 | ||
| 105,6,1.0,35.0,99.999 | ||
| 105,7,1.0,35.0,99.999 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| source_species,target_species,target_esa,source_esa,sputter_factor | ||
| o,h,1,4,0.236 | ||
| o,h,2,4,0.372 | ||
| o,h,3,4,0.898 | ||
| o,h,4,4,0.891 | ||
| o,h,5,4,0.037 | ||
| o,h,5,6,0.32 | ||
| o,h,6,6,0.32 | ||
| o,h,7,6,0.22 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -96,6 +96,36 @@ class LoConstants: | |
| N_SPINS_PER_ESA_LEVEL: int = 4 # Spins per ESA step within one histogram cycle | ||
| N_SPIN_ANGLE_BINS: int = 60 # Number of angular bins within a spin | ||
|
|
||
| # Bootstrap correction settings. The nominal coefficients of the ancillary | ||
| # are scaled by BOOTSTRAP_SCALE before they are applied; the low and high | ||
| # scalings bracket that choice and become the systematic error on the | ||
| # corrected intensity. | ||
| BOOTSTRAP_SCALE: float = 0.5 | ||
| BOOTSTRAP_SCALE_INTENSITY_HIGH: float = 0.25 | ||
| BOOTSTRAP_SCALE_INTENSITY_LOW: float = 1.0 | ||
|
|
||
| # The bootstrap correction of the highest ESA levels needs an ESA level | ||
| # above them to subtract. That virtual "ESA 8" channel has no geometric | ||
| # factor of its own; its intensity is extrapolated from the top two levels | ||
| # with a power law, at this multiple of the top level's center energy. | ||
| ESA_8_ENERGY_RATIO: float = 2.1 | ||
| # Width [pixels] of the neighborhood the spectral index of a pixel that has | ||
| # no measurable one is taken from. | ||
| BOOTSTRAP_SPECTRAL_INDEX_FILTER_SIZE: int = 3 | ||
| # The spectral index to extrapolate with when the map has none to offer. | ||
| BOOTSTRAP_DEFAULT_SPECTRAL_INDEX: float = 1.6 | ||
|
|
||
| # Compton-Getting correction settings. The energy [eV] a hydrogen ENA has | ||
| # in the spacecraft frame purely from the spacecraft's own motion, i.e. | ||
| # 1/2 m_H U^2 at the nominal spacecraft speed of ~30 km/s. The kinematics | ||
| # of the correction are scaled by it. | ||
| CG_ENA_ENERGY_AT_SPACECRAFT_SPEED_EV: float = 4.661 | ||
| # The predictor-corrector that estimates the source spectrum behind the | ||
| # observed one runs until the RMS change in the intensities falls below the | ||
| # tolerance, or the iterations run out. | ||
| CG_MAX_ITERATIONS: int = 20 | ||
| CG_CONVERGENCE_TOLERANCE: float = 0.005 | ||
|
Comment on lines
+126
to
+127
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is a really big PR, so I didn't track it down, but IIRC, the flux correction is totally separate from the CG correction. Doesn't the flux correction get applied to maps in either the s/c or helio-frame? I would expect these variables to be specifically named to flux correction rather than having to do with the CG correction. |
||
|
|
||
| # Nominal spin period [s]. True spin duration is NOT 15 seconds. | ||
| NOMINAL_SPIN_PERIOD_SEC: float = 15.0 | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the CG correction that I implemented, this was computed exactly using SPICE. Probably is just in the noise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes - spice is the correct way to go. Almost none of the steps in the dropbox use spice, and I didn't realize you'd already implemented this. I'll look at your implementation and use it instead.