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

Adding JWST query, has_jwst to data model, JWST Tab to html pages #316

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

charliekilpatrick
Copy link
Collaborator

Fixes # .

Description of the Change

Added jwstImages to mast_query.py. This queries mast for the availability of NIRCam and MIRI data in the same way HST queries work. Data and JPEG URLs are generated directly from the astropy table generated from the query.

Added has_jwst to YSE_Initial_Schema.sql, 0001_initial.py, transient_models.py, transient_serializers.py. This value is initialized in get_archival.py and added to the TransientDict object in TNS_uploads.py.

A JWST tab was added to transient_detail.html. The tab is populated by the values from the get_jwst_image function in view_utils.py, which is invoked in urls.py.

Documentation for the JWST tab was added to the description of HST and Chandra tabs in detail.rst.

Checklist

Please check all that apply to your proposed changes

  • HTML code change
  • Added package dependency
  • Added data
  • Changed django model
  • Documentation change

Additional context

The goal for this change is to make the availability of JWST data part of the data model for transients and provide additional context for those data in the exact same way that has_hst and the HST tab work. The mast_query.py script has been tested and provides the jpegurldict object required by get_jwst_image in view_utils.py. As this script works in exactly the same way as HST queries, no package dependencies or data have been added to YSE-PZ.

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.

1 participant