-
Notifications
You must be signed in to change notification settings - Fork 0
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
V3 #7
Conversation
- use chandra_aca.star_probs.binomial_confidence_interval - use modern Ska package names - use standard main/get_parser pattern (main takes no arguments) - use ska_helpers.logging - use argparse and a better verbosity argument - remove global variable TASK_DATA and add corresponding command-line arg - replaced uses of os.path with pathlib.Path - replace %-style string formats with f-strings
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.
I think we should let github be the archive of the "old" stuff and the new old_ directory is not needed.
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.
This looks great for a "first pass" to modernize this package.
This PR mostly just modernizes this package. This PR does not change the information shown.
It does the following:
pyproject.toml
.make_toc.pl
in python. This is a copy of the one in ac_stat, with small variations. Code could be shared between the two.gui_stat_reports.py
, including:os.path
withpathlib
.Chandra.Time
withCxoTime
.guide_summarize.py
main
/get_parser
structureos.path
withpathlib
.Chandra.Time
withCxoTime
.Testing
Make symbolic links ponting from
$SKA/data/gui_stat_reports
and$SKA/www/ASPECT/gui_stat_reports
to the current data/web directories in SKA. Remove the directories I expect to be modified:Run the commands almost like in the task schedule:
Output