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

Change level 2 extensions #149

Open
5 of 6 tasks
mgalloy opened this issue Feb 8, 2024 · 5 comments
Open
5 of 6 tasks

Change level 2 extensions #149

mgalloy opened this issue Feb 8, 2024 · 5 comments
Assignees
Labels
enhancement New features of the pipeline level 2 Affects level2 products needs testing Draft done, testing needed to complete

Comments

@mgalloy
Copy link
Member

mgalloy commented Feb 8, 2024

Dynamics

The dynamics files extensions should be (and have the names of):

  1. Peak intensity
  2. Enhanced intensity
  3. Corrected LOS velocity — should this have "Corrected" in it? Yes
  4. Line width (FWHM)
  5. Center wavelength intensity (new)

Polarization

The polarization files extensions should be (and have the names of):

  1. Weighted average intensity
  2. Enhanced intensity
  3. Weighted average Q
  4. Weighted average U
  5. Weighted average L
  6. Azimuth — this was already in the code, but not in the processed results yet
  7. Radial azimuth — this was already in the code, but not in the processed results yet
  8. Center wavelength intensity (new)

Quick invert

The quick invert files extensions should be (and have the names of):

  1. Center wavelength intensity
  2. Center wavelength Q
  3. Center wavelength U
  4. Linear polarization
  5. Azimuth
  6. Corrected LOS velocity — should this be "LOS velocity"? No
  7. Line width (FWHM)
  8. Radial azimuth
  9. Peak intensity — this was already in the code, but not in the processed results yet

UCoMP level 2 file reference

For reference, here are the level 2 file extension names in UCoMP (for wave regions providing polarization, otherwise just the first 6 extensions):

  1. Center wavelength intensity
  2. Enhanced intensity
  3. Peak intensity
  4. LOS velocity
  5. Line width (FWHM)
  6. Noise mask
  7. Weighted average I
  8. Weighted average Q
  9. Weighted average U
  10. Weighted average L
  11. Azimuth
  12. Radial azimuth

Questions

  • Should velocity extension in Dynamics file be "Corrected LOS velocity" or "LOS velocity" (matching UCoMP)? Yes
  • Should velocity in quick invert file by "Doppler Velocity" or "LOS velocity" (matching UCoMP)?

Tasks

  • dynamics files
  • polarization files
  • quick inverts
  • Verify above extensions in User Guide
@mgalloy mgalloy added enhancement New features of the pipeline level 2 Affects level2 products labels Feb 8, 2024
@mgalloy mgalloy added this to the Decommissioning reprocessing milestone Feb 8, 2024
@mgalloy mgalloy self-assigned this Feb 8, 2024
@detoma
Copy link
Contributor

detoma commented Feb 8, 2024

Yes, let's call it corrected los velocity.

We will make clear in the data user guide it means corrected for rest wavelength and the rest wavelength subtracted is recorded in the header

@mgalloy mgalloy added the questions Questions for instrument experts label Feb 8, 2024
@detoma
Copy link
Contributor

detoma commented Feb 8, 2024

Let's be consistent and call it Corrected LOS velocity everywhere instead of the using the old COMP name of Doppler velocity.

@detoma
Copy link
Contributor

detoma commented Feb 8, 2024

I updated the user guide based on the new names and extensions above.

@mgalloy
Copy link
Member Author

mgalloy commented Feb 8, 2024

The quick invert files have another extension at the end, "Peak intensity", that was already in the code, but not in the processed results yet.

@mgalloy
Copy link
Member Author

mgalloy commented Feb 8, 2024

The polarization files have Azimuth and Radial azimuth in the code, but not in the processed results yet.

@mgalloy mgalloy added needs testing Draft done, testing needed to complete and removed questions Questions for instrument experts labels Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features of the pipeline level 2 Affects level2 products needs testing Draft done, testing needed to complete
Projects
None yet
Development

No branches or pull requests

2 participants