sot / chandra_aca

Chandra Aspect Camera Tools
https://sot.github.io/chandra_aca
BSD 2-Clause "Simplified" License
0 stars 0 forks source link

Use proseco acq candidate mask for ACA plot (bad_acq_stars) #170

Closed taldcroft closed 3 months ago

taldcroft commented 4 months ago

Description

This fixes an issue where stars from AGASC 1.8 are nearly all plotted in the orange-ish color denoting a star that is not selectable as an acq star. This is currently manifested in starcheck review star catalog plots, making it somewhat difficult to evaluate the star field and catalog.

Requires

Interface impacts

Testing

Unit tests

Independent check of unit tests by Jean

chandra_aca/tests/test_aca_image.py ................. [ 8%] chandra_aca/tests/test_all.py ........................ [ 20%] chandra_aca/tests/test_attitude.py ............................................................. [ 49%] chandra_aca/tests/test_dark_model.py .... [ 51%] chandra_aca/tests/test_drift.py .......................... [ 64%] chandra_aca/tests/test_maude_decom.py .................. [ 73%] chandra_aca/tests/test_planets.py ............... [ 80%] chandra_aca/tests/test_psf.py ... [ 81%] chandra_aca/tests/test_residuals.py ..... [ 84%] chandra_aca/tests/test_star_probs.py ................................ [100%]

======================================== 205 passed in 87.64s (0:01:27)


### Functional tests
<!-- Describe and document results of any functional tests, otherwise leave the text below -->

(ska3) ➜ chandra_aca git:(use-proseco-acq-candidate-mask) env PYTHONPATH=/Users/aldcroft/git/proseco ipython
Python 3.11.8 | packaged by conda-forge | (main, Feb 16 2024, 20:49:36) [Clang 16.0.6 ]

In [2]: from chandra_aca.plot import plot_stars In [3]: plot_stars([284.097466, -37.880561, 311.969530]) Out[3]: <Figure size 1064x1064 with 1 Axes>

#### With this PR
<img width="280" alt="image" src="https://github.com/user-attachments/assets/2859b6f3-e3b3-4bf2-bebe-a55cb4a8fe1b">

#### With current release 4.45.1
<img width="280" alt="image" src="https://github.com/user-attachments/assets/f0b1fef4-d6d8-4fe3-82a1-1675946691ef">

#### Starcheck
On `kady` in flight ska3 environment.
https://icxc.cfa.harvard.edu/aspect/test_review_outputs/chandra_aca/pr170/starcheck.html

export PYTHONPATH=/home/aldcroft/git/chandra_aca:/home/aldcroft/git/proseco cd ~/git/starcheck # @ master ./sandbox_starcheck -dir /data/mpcrit1/mplogs/2024/JUL2524/oflsa

mv files to icxc www area

jeanconn commented 3 months ago

For kicks, I also ran sparkles here https://icxc.cfa.harvard.edu/aspect/test_review_outputs/chandra_aca/pr170/AUG0524A_sparkles/ which looks fine.