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.

Sign up for GitHub

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

Jump to bottom

Guide reporting #197

Merged
merged 14 commits into from
Dec 23, 2018
Merged

Guide reporting #197

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
c712334
First draft of guide reporting
jeanconn Dec 17, 2018
99d2b2b
Lower vmax for spoiler stars to allow seeing them
taldcroft Dec 22, 2018
10b9e38
Fix PEP8
taldcroft Dec 22, 2018
805c2be
Add coding and license
taldcroft Dec 22, 2018
71d6fa4
Get rid of the Tom coordinate labels hack
taldcroft Dec 22, 2018
56dfd7c
Add spoiler mag labels and other tidying
taldcroft Dec 23, 2018
17b0fbf
Refactor plotting into two separate functions
taldcroft Dec 23, 2018
3d56b9e
Consistent naming (guide is guide cand, star is field star)
taldcroft Dec 23, 2018
b94e73c
Auto-format COLOR1 to 2 decimal places
taldcroft Dec 23, 2018
f7543db
Remove MAG_ACA and MAG_ACA_ERR from table
taldcroft Dec 23, 2018
dede057
Include guide_index_template.html in package data
taldcroft Dec 23, 2018
1809a71
Add guide star catalog to report
taldcroft Dec 23, 2018
f09e3a2
Put back MAG_ACA_ERR
taldcroft Dec 23, 2018
15a0138
Add ASPQ1 to guide & guide candidates table
taldcroft Dec 23, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion proseco/guide_report.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
# Do reporting for at-most MAX_CAND candidates
MAX_CAND = 50
COLS = ['id', 'stage', 'forced',
'mag', 'mag_err', 'MAG_ACA', 'MAG_ACA_ERR',
'mag', 'mag_err',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

MAG_ACA_ERR is used directly in the spoiler tests. I thought it was somewhat useful to have it here because I found myself looking it up for the cases where it is not presently reasonable (bright candidates). But I suppose if the ids are links to the kadi star lookup that's fine.

'yang', 'zang', 'row', 'col', 'COLOR1']
FILEDIR = Path(__file__).parent
APL = AcaPsfLibrary()
Expand Down