-
Notifications
You must be signed in to change notification settings - Fork 1
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
Data product docs #37
base: order-positions
Are you sure you want to change the base?
Conversation
---------------------------------------- | ||
|
||
.. image:: http://img.shields.io/badge/powered%20by-AstroPy-orange.svg?style=flat | ||
:target: http://www.astropy.org | ||
:alt: Powered by Astropy Badge | ||
|
||
BANZAI-FLOYDS is the Las Cumbres Observatory pipeline to process `FLOYDS <https://lco.global/observatory/instruments/floyds/>` data. We currently have two FLOYDS insturments in operation: one on Haleakala in Hawaii and one at Siding Springs in Australia. The FLOYDS spectrographs are low-resolution, workhorse spectrographs for supernovae and other explosive transients. FLOYDS has a resolution of R~500 and covers from 320nm - 1000nm. FLOYDS is double dispersed so the images of the slit are curved on the detector (similar to eschelle spectrographs). The sky lines are also tilted to better sample the line spread funciton to enable better sky subtraction. |
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.
instruments typo
function typo
are 'order', 'wavelength', 'center', 'center_error', 'sigma', and 'sigma_error'. These points are estimated by taking | ||
slices in the y-direction and stepping along the dispersion direction. | ||
|
||
- 'PRFOILE' Extension: This extension has a 2-D image of the profile weights. This is for convenience so the user can |
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.
PROFILE rather than PRFOILE?
|
||
Profile Fitting | ||
--------------- | ||
To estimte the center of the profile that we want to extract, we perform a match filter with a Gaussian, fixing Gaussian's |
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.
Typo in estimate
---------------------------------------- | ||
Flux standards are observed periodically and are calibrated with known reference fluxes. The README in the | ||
banzai_floyds/data/standards directory provides more details. The telluric features are treated as a multiplicative | ||
correction and are based on the same flux standard observations. The initial sensitivity function to disenangle the |
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.
Typo in disentangle
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.
A few typos but looks good.
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'm not sure exactly what's going on, but when I try to build the docs myself locally using make html
in the docs directory, I get the following error:
Configuration error:
There is a programmable error in your configuration file:
Traceback (most recent call last):
File "/home/jchatelain/.local/lib/python3.12/site-packages/sphinx/config.py", line 358, in eval_config_file
exec(code, namespace) # NoQA: S102
^^^^^^^^^^^^^^^^^^^^^
File "/home/jchatelain/git/banzai-floyds/docs/conf.py", line 79, in <module>
import_module(setup_cfg['name'])
File "/usr/lib64/python3.12/importlib/__init__.py", line 90, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
File "<frozen importlib._bootstrap>", line 1324, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'banzai_floyds'
make: *** [Makefile:44: html] Error 2
I'm not sure what the import_module()
method does, but it seems to be having some issues here or I'm doing something wrong.
---------------------------------------- | ||
|
||
.. image:: http://img.shields.io/badge/powered%20by-AstroPy-orange.svg?style=flat | ||
:target: http://www.astropy.org | ||
:alt: Powered by Astropy Badge | ||
|
||
BANZAI-FLOYDS is the Las Cumbres Observatory pipeline to process `FLOYDS <https://lco.global/observatory/instruments/floyds/>` data. We currently have two FLOYDS insturments in operation: one on Haleakala in Hawaii and one at Siding Springs in Australia. The FLOYDS spectrographs are low-resolution, workhorse spectrographs for supernovae and other explosive transients. FLOYDS has a resolution of R~500 and covers from 320nm - 1000nm. FLOYDS is double dispersed so the images of the slit are curved on the detector (similar to eschelle spectrographs). The sky lines are also tilted to better sample the line spread funciton to enable better sky subtraction. |
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.
you need a "_" after the link for it to work:
`FLOYDS <https://lco.global/observatory/instruments/floyds/>`_ data
---------------------------------------- | ||
|
||
.. image:: http://img.shields.io/badge/powered%20by-AstroPy-orange.svg?style=flat | ||
:target: http://www.astropy.org | ||
:alt: Powered by Astropy Badge | ||
|
||
BANZAI-FLOYDS is the Las Cumbres Observatory pipeline to process `FLOYDS <https://lco.global/observatory/instruments/floyds/>` data. We currently have two FLOYDS insturments in operation: one on Haleakala in Hawaii and one at Siding Springs in Australia. The FLOYDS spectrographs are low-resolution, workhorse spectrographs for supernovae and other explosive transients. FLOYDS has a resolution of R~500 and covers from 320nm - 1000nm. FLOYDS is double dispersed so the images of the slit are curved on the detector (similar to eschelle spectrographs). The sky lines are also tilted to better sample the line spread funciton to enable better sky subtraction. |
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 don't think we should limit ourselves to explosive transients. A non-negligible portion of Floyds spectra go towards classifying asteroids, for example.
|
||
The dataflow and infrastructure of this pipeline relies heavily on `BANZAI | ||
<https://github.com/lcogt/banzai>`_, enabling this repo to focus on analysis that is specific to slit spectroscopy. | ||
|
||
Installation | ||
------------ | ||
``` |
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.
This is improperly formatted.
Use
.. code-block:: sh
pip install .
Instead (Make sure you have an empty space after too.)
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.
You can also get away with just
::
pip install .
I think...
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.
Below this in the license, if you want to include the actual copyright symbol, you can do
This project is Copyright |copy| cmccully and licensed under ...
.. |copy| unicode:: 0xA9 .. copyright sign
Data Products | ||
============= | ||
The banzai-floyds data products are split into a variety of files so that users only need to download the files | ||
they need for their analysis. All intermediate products are available to users to enable debugging of any reduction |
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.
" the files required for their analysis." sounds better
Extracted spectra are in files with the '1d' suffix and 'SPECTRUM' OBSTYPE, following a naming convention like | ||
"ogg2m001-en06-20250111-0056-e91-1d.fits.fz". The extracted files are multi-extension fits files. | ||
|
||
- 'SPECTRUM' Extension: This extension includes what is considered the final reduction of the spectrum. |
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 would emphasize the extension name by making it bold
|
||
Fringe Frames | ||
------------- | ||
Combined (stacked) lamp flat exposures are used to correct for fringing and have the '-lampflat' filename suffix. |
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.
The fringe frames have the -lampflat
filename suffix?
---------- | ||
HgAr exposures are used for wavelength calibration. These frames have an 'a91' filename suffix and an OBSTYPE of ARC. The | ||
'SCI', 'BPM', 'ERR', and 'ORDER_COEFFS' are the same as the 2-D spectroscopic images. The only difference is that the | ||
'WAVELENGTHS' extension is derived from this frame rather than being copied in. are the same as the 2-D spectroscopic images. Science frames reference the arc that provided the WAVELENGTH extension via the L1IDARC header keyword. The |
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.
Remove "are the same as the 2-D spectroscopic images."
We use a HgAr arc lamp frame to find the wavelength solution. We use the arc frame taken closest to the science | ||
exposure, preferably in the same block. If an arc in the same block does not exist, we fall back to other arcs taken | ||
under the same program, and finally to any arc taken in the same configuration that is now public. | ||
We employ a matched filter model that uses lines in banzai_floyds.arc_lines as the reference spectrum. |
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.
`banzai_floyds.arc_lines`
|
||
Flux Calibration and Telluric Correction | ||
---------------------------------------- | ||
Flux standards are observed periodically and are calibrated with known reference fluxes. The README in the |
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.
This 'README' should probably be a link.
Flux Calibration and Telluric Correction | ||
---------------------------------------- | ||
Flux standards are observed periodically and are calibrated with known reference fluxes. The README in the | ||
banzai_floyds/data/standards directory provides more details. The telluric features are treated as a multiplicative |
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.
`banzai_floyds/data/standards`
This PR adds documentation explaining the data format of the reduction products of the pipeline.