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

Errors in procedure documentation - pixel selection cases for imaging in "stx_sum_pixel_data.pro" #214

Open
PaipaPsyche opened this issue Jun 6, 2024 · 3 comments
Labels
bug Something isn't working reviewed 2024-10

Comments

@PaipaPsyche
Copy link

The procedure "stx_plot_selected_time_range.pro" uses a switch-case loop to determine whether to use the top, bottom or all pixels for imaging ('sumcase' parameter). However the options defined in the documentation of "stx_sum_pixel_data.pro" do not coincide entirely with the functional cases. I don't know if this happens in other files.

@PaipaPsyche PaipaPsyche added the bug Something isn't working label Jun 6, 2024
@samaloney
Copy link
Collaborator

@paolomassa / @FredSchuller thoughts, I quickly looked at the code but don't see a difference/problem?

@paolomassa
Copy link
Collaborator

Not sure what is the problem here. @PaipaPsyche could you please give us more information about this bug? Thanks!

@PaipaPsyche
Copy link
Author

I used a wrong tag, more than a bug I think is misleading documentation: I wanted to use just the bottom pixels for imaging, so I tried using the 'BOTTOM' key in the sumacase parameter as suggested by the documentation of "stx_sum_pixel_data.pro"(line 25). But it doesn't work, as the case 'BOTTOM' is not contemplated as an option in the switch case of "stx_plot_selected_time_range.pro" (line 116), the expected keyword for using bottom pixels is 'BOT'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working reviewed 2024-10
Projects
None yet
Development

No branches or pull requests

3 participants