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

Level 2 data products, plots, and movies #151

Open
3 of 4 tasks
detoma opened this issue Mar 5, 2024 · 4 comments
Open
3 of 4 tasks

Level 2 data products, plots, and movies #151

detoma opened this issue Mar 5, 2024 · 4 comments
Assignees
Labels
level 2 Affects level2 products

Comments

@detoma
Copy link
Contributor

detoma commented Mar 5, 2024

Some changes to the level 2 products.

Tasks

  • Radial azimuth plots should go from -90 to 90 degrees, not -50 to 50 degrees.
  • FWHM should use same colors table than UCOMP and same min and max, i.e. 54 and 74
  • Velocity colors table should the same as UCoMP (we think it is). Let's keep the min and max at -10 km/s +10 km/s for now.
  • Need to use NaNs instead of 0s for masking bad data to prevent weird boundaries seen in the images below.
@detoma detoma added this to the Decommissioning reprocessing milestone Mar 5, 2024
@detoma
Copy link
Contributor Author

detoma commented Mar 5, 2024

FWHM should use same colors table than UCOMP and same min and max, i.e. 54 and 74

Velocity colors table should the same as UCOMP (we think it is). Let's keep the min and max at -10km/s +1km/s for now.

@detoma
Copy link
Contributor Author

detoma commented Mar 5, 2024

Velocity maps do not look correct. There are strange boundaries that do not look realistic.
Screenshot 2024-03-05 at 11 24 40 AM

This is what my computation of the velocity map looks like:

Screenshot 2024-03-05 at 1 51 59 PM

@detoma detoma assigned detoma and mgalloy and unassigned detoma Mar 5, 2024
@detoma
Copy link
Contributor Author

detoma commented Mar 5, 2024

There are multiple issues in the velocity computation. I will re-write comp_l2_analytical.pro (see issue #136) to fix them. This should address the strange velocity maps posted above.

@detoma detoma changed the title level 2 plots and movies level 2 data products, plots, and movies Mar 5, 2024
@detoma detoma self-assigned this Mar 6, 2024
@mgalloy mgalloy changed the title level 2 data products, plots, and movies Level 2 data products, plots, and movies Mar 6, 2024
@mgalloy mgalloy added the level 2 Affects level2 products label Mar 6, 2024
@mgalloy
Copy link
Member

mgalloy commented Mar 11, 2024

The doppler color tables for CoMP and UCoMP are not found exactly the same way, but are equivalent.

IDL> print, ucomp_r - comp_r
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0
IDL> print, ucomp_g - comp_g
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0
IDL> print, ucomp_b - comp_b
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
   0   0   0   0   0   0

@mgalloy mgalloy added needs testing Draft done, testing needed to complete and removed needs testing Draft done, testing needed to complete labels Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level 2 Affects level2 products
Projects
None yet
Development

No branches or pull requests

2 participants