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

Added epoch 13 support #293

Merged
merged 2 commits into from
Oct 7, 2021
Merged

Added epoch 13 support #293

merged 2 commits into from
Oct 7, 2021

Conversation

ajstewart
Copy link
Collaborator

@ajstewart ajstewart commented Oct 7, 2021

  • Changed all previous 12 data files and code entries to 13.
  • Added new data files for epoch 12.
  • Updated VAST_PILOT.stmoc.fits.
  • Added code warnings about epoch 12.
  • Added docs warnings about epoch 12.

Fixes #279.

@ddobie I'll share with you offline what I did to use askap_surveys, it can be documented whenever #292 is done.

- Changed all previous 12 data files and code entries to 13.
- Added new data files for epoch 12.
- Updated `VAST_PILOT.stmoc.fits`.
- Added code warnings about epoch 12.
- Added docs warnings about epoch 12.
@ajstewart ajstewart added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 7, 2021
@ajstewart ajstewart self-assigned this Oct 7, 2021
@ajstewart
Copy link
Collaborator Author

I've added warnings to the the init functions of the MOC and Query class. I decided to not put it on the Survey classes as these are used internally by the code so you'd have them popping up everywhere.

Screen Shot 2021-10-07 at 14 28 56

Screen Shot 2021-10-07 at 14 43 24

@ajstewart ajstewart requested a review from ddobie October 7, 2021 13:44
@ajstewart ajstewart marked this pull request as ready for review October 7, 2021 13:44
@ajstewart ajstewart merged commit 9d43acc into dev Oct 7, 2021
@ajstewart ajstewart deleted the iss279-epoch13-support branch October 7, 2021 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for epoch 13
2 participants