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

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

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

taldcroft
Copy link
Member

@taldcroft taldcroft commented Jul 25, 2024

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

  • The filtering mask in proseco requires two new AGASC columns compared to the previous chandra_aca.plot code: MAG_ACA and COLOR1.
  • The filtering in proseco is somewhat different from the previous code, so there will be changes in the color of plotted stars. In general the changes are for the better and accurately reflect what proseco is using in star selection.

Testing

Unit tests

  • Mac

Independent check of unit tests by Jean

  • Linux
ska3-jeanconn-fido> export PYTHONPATH=/home/jeanconn/git/proseco
ska3-jeanconn-fido> git rev-parse HEAD
ab418934559da69f8a3afa55433be4d33fa596a2
ska3-jeanconn-fido> pytest
============================================== test session starts ==============================================
platform linux -- Python 3.11.8, pytest-7.4.4, pluggy-1.4.0
rootdir: /proj/sot/ska/jeanproj/git
configfile: pytest.ini
plugins: anyio-4.3.0, timeout-2.2.0
collected 205 items                                                                                             

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

(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

image

With current release 4.45.1

image

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

@taldcroft taldcroft requested review from javierggt and jeanconn July 25, 2024 16:48
Copy link
Contributor

@jeanconn jeanconn left a comment

Choose a reason for hiding this comment

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

I think this should also get a functional test via starcheck.

@jeanconn
Copy link
Contributor

jeanconn commented Aug 1, 2024

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

@taldcroft taldcroft merged commit 3425163 into master Aug 1, 2024
2 checks passed
@taldcroft taldcroft deleted the use-proseco-acq-candidate-mask branch August 1, 2024 14:54
@javierggt javierggt mentioned this pull request Nov 7, 2024
@javierggt javierggt mentioned this pull request Nov 19, 2024
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