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

Auto yaml #83

Merged
merged 13 commits into from
Feb 18, 2021
Merged

Auto yaml #83

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
1c11f97
refactor methods so all observations are kept in the obs_stats table,…
javierggt Feb 10, 2021
93feded
Changed the logic by which observations were skipped if they are alre…
javierggt Feb 10, 2021
4a5b432
changed --data-root argument to --output-dir so it agrees with update…
javierggt Feb 10, 2021
974e39f
Added functionality to automatically save a YAML file with observatio…
javierggt Feb 10, 2021
3a7aff7
Fixes in report
javierggt Feb 10, 2021
5f1e23a
renamed n_obs_bad_new to n_obs_bad_fail
javierggt Feb 10, 2021
c71ca09
report plot cleanup so it always displays something even if things fa…
javierggt Feb 10, 2021
e478641
fixup
javierggt Feb 10, 2021
0736f0e
split obsids_fail into obsids_fail + obsids_suspect
javierggt Feb 10, 2021
86a4fc5
small fix
javierggt Feb 17, 2021
02ae39f
fixed another error due to rebases
javierggt Feb 17, 2021
55760b3
add templates which seem to have been forgotten
javierggt Feb 17, 2021
87b12f8
fix: for the purpose of agasc_stats (and the report), count only unex…
javierggt Feb 17, 2021
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
8 changes: 4 additions & 4 deletions agasc/scripts/update_obs_status.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ def update_obs_table(filename, obs_status_override, dry_run=False, create=False)
obs_status = table.Table(dtype=obs_dtype)

if not dry_run:
obs_status.write(filename, format='hdf5', path='obs', append=True, overwrite=True)
obs_status.write(str(filename), format='hdf5', path='obs', append=True, overwrite=True)
save_version(filename, 'obs')
else:
logger.info('dry run, not saving anything')
Expand Down Expand Up @@ -283,7 +283,7 @@ def get_parser():
description=__doc__,
parents=[get_obs_status_parser()]
)
parse.add_argument("--data-root",
parse.add_argument("--output-dir",
default='.',
type=Path,
help="Directory containing agasc_supplement.h5 (default='.')")
Expand All @@ -309,7 +309,7 @@ def update(args):

if status['obs']:
update_obs_table(
args.data_root / 'agasc_supplement.h5',
args.output_dir / 'agasc_supplement.h5',
status['obs'],
dry_run=args.dry_run
)
Expand All @@ -318,7 +318,7 @@ def update(args):
bad_star_ids, bad_star_source = zip(*status['bad'].items())
add_bad_star(bad_star_ids,
bad_star_source,
args.data_root / 'agasc_supplement.h5',
args.output_dir / 'agasc_supplement.h5',
dry_run=args.dry_run)
return [o[1] for o in status['obs']]

Expand Down
Loading