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

Add additions from CADC work #6

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Conversation

Nat1405
Copy link
Contributor

@Nat1405 Nat1405 commented Jul 28, 2020

This pull request is to merge in bugfixes to Nifty4Gemini completed at the Canadian Astronomy Data Centre (CADC) during summer 2020. It maintains full backwards compatibility with the old Nifty4Gemini, with the addition of a new command line -s <datasource> flag which allows data to be either downloaded from the Gemini Observatory archive or the CADC's Gemini Science Archive. It will eventually include bug fixes discovered by processing the entire Gemini NIFS archive with Nifty4Gemini.

Bad information in the config.cfg (a science directory that didn't exist) was causing nifsMerge to fail. I added better exception handling so hint to users why it is failing.
Users can now specify -c (eg; runNifty nifsPipeline -c ...)
to download raw data from the Canadian Astronomy Data Centre.
This has been tested to work from the interactive config session
(runNifty nifsPipeline -i) and the fully automatic mode
(runNifty nifsPipeline -c -f <PROGRAM ID>).
Rather than just choosing between Gemini and CADC downloads,
--data-source adds support for more than just two data sources.
To add support for a new archive at minimum nifsSort.py needs
changed.
There were problems with finding the config file.
Adds CADC download option, plus fixes bugs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant