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
jp-1690 calspec3 cube_build clean up #5347
jp-1690 calspec3 cube_build clean up #5347
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5347 +/- ##
==========================================
+ Coverage 52.67% 52.78% +0.10%
==========================================
Files 407 409 +2
Lines 36907 37027 +120
Branches 5725 5745 +20
==========================================
+ Hits 19442 19545 +103
- Misses 16227 16239 +12
- Partials 1238 1243 +5
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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 OK overall, just a couple minor comments. Oh, and of course it needs a change log entry eventually.
jwst/cube_build/blot_cube_build.py
Outdated
# ********************************************************************** | ||
|
||
def blot_info(self): | ||
""" Prints the basic paramters of the blot image and median sky cube | ||
""" | ||
log.info('Information on Blotting') | ||
log.info('Working with instrument %s %s', self.instrument, | ||
self.detector) | ||
log.info('Working with instrument %s ', self.instrument) |
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.
Minor nit, but could remove unused white space after %s
jwst/cube_build/ifu_cube.py
Outdated
# If all the data is flagged as DO_NOT_USE - not common then log warning and skip data | ||
nn = wave.size | ||
no_data = False | ||
if nn == 0: |
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.
Is nn
used anywhere else? If not, why not just consolidate these lines into:
if wave.size == 0:
no_data = True
else:
no_data = False
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 OK to me. Still needs a change log entry before merging.
Clean up of making single cubes and blotted cubes.