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

obs table initialized with incorrect types #63

Closed
taldcroft opened this issue Jan 28, 2021 · 0 comments · Fixed by #76
Closed

obs table initialized with incorrect types #63

taldcroft opened this issue Jan 28, 2021 · 0 comments · Fixed by #76

Comments

@taldcroft
Copy link
Member

Starting from a supplement that has no pre-existing obs dataset:

(ska3) ➜  agasc git:(update-script-fixes) python agasc/scripts/update_obs_status.py --obs 46686 --agasc-id=175379112 --status=bad --comments="ACA high background event"
INFO:agasc.supplement:updating "obs" table in agasc_supplement.h5
INFO:agasc.supplement:Appending agasc_id='175379112', obsid='46686', status={'ok': 0, 'comments': 'ACA high background event'}
(ska3) ➜  agasc git:(update-script-fixes) ipython
Python 3.8.3 (default, Jul  2 2020, 11:26:31) 
Type 'copyright', 'credits' or 'license' for more information
IPython 7.16.1 -- An enhanced Interactive Python. Type '?' for help.

In [1]: astro                                                                                                                                      
astropy=4.0.1.post1

In [2]: t = Table.read('agasc_supplement.h5', path='obs')                                                                                          

In [3]: t                                                                                                                                          
Out[3]: 
<Table length=1>
obsid   agasc_id   ok            comments        
bytes5   bytes9  uint64          bytes25         
------ --------- ------ -------------------------
 46686 175379112      0 ACA high background event
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant