-
Notifications
You must be signed in to change notification settings - Fork 103
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
Collate 1d updates #1649
Collate 1d updates #1649
Conversation
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## develop #1649 +/- ##
========================================
Coverage 40.98% 40.99%
========================================
Files 189 189
Lines 43055 43092 +37
========================================
+ Hits 17646 17664 +18
- Misses 25409 25428 +19
|
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.
Looks good; one minor edit from me.
pypeit/core/collate.py
Outdated
"""Build a list of SourceObjects from a list of spec1d files. There will be one SourceObject per | ||
SpecObj in the resulting list (i.e. no combining or collating is done by this method). | ||
|
||
Args: | ||
spec1d_files (list of str): List of spec1d filenames | ||
spec1d_files (list of :obj:`pypeit.specobjs.SpecObjs`): List of SpecObjs objects to build from. |
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.
specobjs_list
?
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.
Looks good. Thank you @badpandabear!
pypeit/scripts/collate_1d.py
Outdated
- **missing_archive_msgs** (:obj:`list`): A list of messages | ||
explaining why some source objects were excluded. | ||
|
||
**excluded_messages** (:obj:`list`): A list of messages explaining why some source objects were excluded. |
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 here it's missing the -
for the bullet list
dev-suite run:
The failed unit tests were later fixed by |
Updates to pypeit_collate_1d for ADAP 2019. Changes include: