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

Improve functionality for Gaia-only targets. #673

Merged
merged 32 commits into from Jan 29, 2021
Merged

Conversation

geordie666
Copy link
Contributor

This PR includes a range of improved functionality for Gaia-only targets:

  • Deredden the Gaia colors when selecting Gaia-only standards.
    • Include a general function to de-extinct Gaia magnitudes.
    • Add E(B-V) as an EBV column in the supplemental targets for downstream spectrophotometric calibrations.
  • Add a flavor property to the bitmask for the secondary targets.
    • This flavor can be "SPARE", "DEDICATED", "SSV", "QSO" or "TOO".
    • Also add a unit test to check the allowed values of flavor.
  • Update the priorities for "dedicated" secondary targets.
    • This is facilitated by the fact that such targets can be distinguished from "spare" targets using flavor.
  • Include PHOTSYS of "G" for targets that are selected using only Gaia.
  • Fix some minor bugs in directory structures and columns in pixweight files that arose due to changes in the data model.
  • Add code to scrape the Gaia EDR3 flat (csv) files from the web.
    • Also code to reformat them as FITS files, and by HEALPixel.

…f requested, which we haven't ever, yet, in desitarget)
…Project targets, as then we can top up dedicated programs with KP targets
@geordie666
Copy link
Contributor Author

I need to run new targets to go back on-sky tomorrow. So, I'll merge this and make a new tag after tests pass (likely early tomorrow morning).

@geordie666 geordie666 merged commit 6ebeca6 into master Jan 29, 2021
@geordie666 geordie666 deleted the ADM-gaia-edr3 branch January 29, 2021 13:50
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